From ad71a0fc2807f19a61df75bedc53b2ca9af0da49 Mon Sep 17 00:00:00 2001 From: "jiayl@chromium.org" Date: Thu, 19 Sep 2013 01:24:57 +0000 Subject: Removing the IsMailboxNameValid check in MailboxManager::ConsumeTexture for perf. According to piman, the check is unnecessary as the mailbox is already in a map. BUG= Review URL: https://chromiumcodereview.appspot.com/23619079 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224017 0039d316-1c4b-4281-b951-d872f2087c98 --- gpu/command_buffer/service/mailbox_manager.cc | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'gpu') diff --git a/gpu/command_buffer/service/mailbox_manager.cc b/gpu/command_buffer/service/mailbox_manager.cc index a569bb5..bb2f726 100644 --- a/gpu/command_buffer/service/mailbox_manager.cc +++ b/gpu/command_buffer/service/mailbox_manager.cc @@ -40,14 +40,12 @@ void MailboxManager::GenerateMailboxName(MailboxName* name) { Texture* MailboxManager::ConsumeTexture(unsigned target, const MailboxName& name) { - if (!IsMailboxNameValid(name)) - return NULL; - MailboxToTextureMap::iterator it = mailbox_to_textures_.find(TargetName(target, name)); if (it == mailbox_to_textures_.end()) return NULL; + DCHECK(IsMailboxNameValid(name)); return it->second->first; } -- cgit v1.1