diff options
| author | jiayl@chromium.org <jiayl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-19 01:24:57 +0000 |
|---|---|---|
| committer | jiayl@chromium.org <jiayl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-19 01:24:57 +0000 |
| commit | ad71a0fc2807f19a61df75bedc53b2ca9af0da49 (patch) | |
| tree | 9c324f0a289c441d6fde777467f081a93102f9f2 /gpu | |
| parent | e2035da570fdfd36cbed3a3cdfcf5b8c14a4e233 (diff) | |
| download | chromium_src-ad71a0fc2807f19a61df75bedc53b2ca9af0da49.zip chromium_src-ad71a0fc2807f19a61df75bedc53b2ca9af0da49.tar.gz chromium_src-ad71a0fc2807f19a61df75bedc53b2ca9af0da49.tar.bz2 | |
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
Diffstat (limited to 'gpu')
| -rw-r--r-- | gpu/command_buffer/service/mailbox_manager.cc | 4 |
1 files changed, 1 insertions, 3 deletions
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; } |