Coverity fixlet.

chrome/browser/extensions/extension_info_map.cc

Index: chrome/browser/extensions/extension_info_map.cc
diff --git a/chrome/browser/extensions/extension_info_map.cc b/chrome/browser/extensions/extension_info_map.cc
index 39357c11e1ec20ed293ebd9599940c93253c61e4..afe20fa059e660ce91166dd8f93a011a99b28646 100644
--- a/chrome/browser/extensions/extension_info_map.cc
+++ b/chrome/browser/extensions/extension_info_map.cc
@@ -127,7 +127,9 @@ bool ExtensionInfoMap::SecurityOriginHasAPIPermission(
     extensions::APIPermission::ID permission) const {
   if (origin.SchemeIs(chrome::kExtensionScheme)) {
     const std::string& id = origin.host();
-    return extensions_.GetByID(id)->HasAPIPermission(permission) &&
+    const Extension* extension = extensions_.GetByID(id);
+    DCHECK(extension != NULL);

[asargent_no_longer_on_chrome, 2012/07/25 19:22:15]

It might be better to make this a CHECK instead of DCHECK, since we'll get crash
on next line anyway if |extension| is null. 

[Danh Nguyen, 2012/07/25 20:25:03]

Done.

+    return extension->HasAPIPermission(permission) &&
         process_map_.Contains(id, process_id);
   }