Bookmark Manager Drag and Drop backend....

chrome/browser/extensions/extension_dom_ui.cc

Index: chrome/browser/extensions/extension_dom_ui.cc
===================================================================
--- chrome/browser/extensions/extension_dom_ui.cc	(revision 39450)
+++ chrome/browser/extensions/extension_dom_ui.cc	(working copy)
@@ -7,12 +7,14 @@
 #include "base/string_util.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_list.h"
+#include "chrome/browser/extensions/extension_bookmark_manager_api.h"
 #include "chrome/browser/extensions/extensions_service.h"
 #include "chrome/browser/pref_service.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/renderer_host/render_widget_host_view.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/common/bindings_policy.h"
+#include "chrome/common/chrome_switches.h"
 #include "chrome/common/url_constants.h"
 
 namespace {
@@ -48,12 +50,23 @@
       new ExtensionFunctionDispatcher(render_view_host, this, url));
 }
 
+void ExtensionDOMUI::ResetExtensionBookmarkManagerEventRouter() {
+  // TODO(arv): Move to ExtensionHost?

[Erik does not do reviews, 2010/02/19 23:47:17]

ExtensionsService - but maybe I misunderstood what was going on here.  Does each
DOMUI need its own router?  And is it specific to tab contents?  I was used to
the other event routers which were more global.  If that's the case, then I
think it's OK to keep it here.

[arv (Not doing code reviews), 2010/02/20 00:04:31]

Yes, each ExtensionDOMUI tab contents needs its own router. I was thinking that
maybe the extension service could be used to only dispatch to the view that the
user was dragging over.

After looking at ExtensionsService I think the current solution is clearer.

+  if (CommandLine::ForCurrentProcess()->HasSwitch(
+      switches::kEnableExperimentalExtensionApis)) {
+    extension_bookmark_manager_event_router_.reset(
+        new ExtensionBookmarkManagerEventRouter(GetProfile(), tab_contents()));
+  }
+}
+
 void ExtensionDOMUI::RenderViewCreated(RenderViewHost* render_view_host) {
   ResetExtensionFunctionDispatcher(render_view_host);
+  ResetExtensionBookmarkManagerEventRouter();
 }
 
 void ExtensionDOMUI::RenderViewReused(RenderViewHost* render_view_host) {
   ResetExtensionFunctionDispatcher(render_view_host);
+  ResetExtensionBookmarkManagerEventRouter();
 }
 
 void ExtensionDOMUI::ProcessDOMUIMessage(const std::string& message,