[Extensions] Clean up the handling of ExtensionHostMsg_Request

chrome/browser/extensions/chrome_extension_function_details.cc

Index: chrome/browser/extensions/chrome_extension_function_details.cc
diff --git a/chrome/browser/extensions/chrome_extension_function_details.cc b/chrome/browser/extensions/chrome_extension_function_details.cc
index d80c0812715fb793eb5156b810c32025a4f8c3ef..614511fcbb1f00a2ec65628ba84cabb33260da7d 100644
--- a/chrome/browser/extensions/chrome_extension_function_details.cc
+++ b/chrome/browser/extensions/chrome_extension_function_details.cc
@@ -54,7 +54,7 @@ Browser* ChromeExtensionFunctionDetails::GetCurrentBrowser() const {
   // If the delegate has an associated browser, return it.
   if (function_->dispatcher()) {
     extensions::WindowController* window_controller =
-        function_->dispatcher()->delegate()->GetExtensionWindowController();
+        function_->dispatcher()->GetExtensionWindowController();

[not at google - send to devlin, 2015/06/10 20:48:12]

It would be nice if this were just function_->GetWindowController() (likewise
GetVisibleWebContents), and the dispatcher() accessor on ExtensionFunction
entirely removed. But, not necessarily right now.

[Devlin, 2015/06/10 20:59:20]

Agreed - I intend to try and clean up ExtensionFunction's dependence on
Dispatcher at some point, but that's a later patch.

     if (window_controller) {
       Browser* browser = window_controller->GetBrowser();
       if (browser)
@@ -93,7 +93,7 @@ ChromeExtensionFunctionDetails::GetExtensionWindowController() const {
   // If the delegate has an associated window controller, return it.
   if (function_->dispatcher()) {
     extensions::WindowController* window_controller =
-        function_->dispatcher()->delegate()->GetExtensionWindowController();
+        function_->dispatcher()->GetExtensionWindowController();
     if (window_controller)
       return window_controller;
   }