Fix callback for chrome.tabs.update with javascript URLs.

chrome/browser/extensions/extension_tabs_module.h

Index: chrome/browser/extensions/extension_tabs_module.h
diff --git a/chrome/browser/extensions/extension_tabs_module.h b/chrome/browser/extensions/extension_tabs_module.h
index 944de1c7382c2a8f044a405083464ac82cf22337..726f7fe89e7519141c2e69bb55b9768bc5f07d0a 100644
--- a/chrome/browser/extensions/extension_tabs_module.h
+++ b/chrome/browser/extensions/extension_tabs_module.h
@@ -18,11 +18,13 @@
 
 class BackingStore;
 class SkBitmap;
-
+class TabStripModel;

[Mihai Parparita -not on Chrome, 2012/01/30 16:33:00]

Nit: This can be removed now.

[jstritar, 2012/01/30 16:40:23]

Done.

 namespace base {
 class DictionaryValue;
 }  // namespace base
-
+namespace content {
+class WebContents;
+}  // namespace content
 // Windows
 class GetWindowFunction : public SyncExtensionFunction {
   virtual ~GetWindowFunction() {}
@@ -112,10 +114,13 @@ class UpdateTabFunction : public AsyncExtensionFunction,
  private:
   virtual ~UpdateTabFunction() {}
   virtual bool RunImpl() OVERRIDE;
+  virtual void WebContentsDestroyed(content::WebContents* tab) OVERRIDE;
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
   void OnExecuteCodeFinished(int request_id,
                              bool success,
                              const std::string& error);
+  void PopulateResult();
+  content::WebContents* web_contents_;

[Mihai Parparita -not on Chrome, 2012/01/30 16:33:00]

Nit: mind adding an empty line between the methods and members (to aid
readability)?

[jstritar, 2012/01/30 16:40:23]

Done.

   DECLARE_EXTENSION_FUNCTION_NAME("tabs.update")
 };
 class MoveTabsFunction : public SyncExtensionFunction {