Avoid calling ExecuteCode() on a NULL TabContents. The problem was an extra s...

chrome/browser/extensions/execute_code_in_tab_function.cc

Index: chrome/browser/extensions/execute_code_in_tab_function.cc
===================================================================
--- chrome/browser/extensions/execute_code_in_tab_function.cc	(revision 42300)
+++ chrome/browser/extensions/execute_code_in_tab_function.cc	(working copy)
@@ -128,9 +128,12 @@
 bool ExecuteCodeInTabFunction::Execute(const std::string& code_string) {
   TabContents* contents = NULL;
   Browser* browser = NULL;
-  if (!ExtensionTabUtil::GetTabById(execute_tab_id_, profile(),
-                                    include_incognito(), &browser, NULL,
-                                    &contents, NULL) && contents && browser) {
+
+  bool success = ExtensionTabUtil::GetTabById(
+      execute_tab_id_, profile(), include_incognito(), &browser, NULL,
+      &contents, NULL) && contents && browser;
+
+  if (!success) {
     SendResponse(false);
     return false;
   }