GuestView: Move GuestViewManager extension dependencies to ExtensionsGuestViewManagerDelegate

extensions/browser/guest_view/web_view/web_view_apitest.cc

Index: extensions/browser/guest_view/web_view/web_view_apitest.cc
diff --git a/extensions/browser/guest_view/web_view/web_view_apitest.cc b/extensions/browser/guest_view/web_view/web_view_apitest.cc
index 61b87bde5ae18bed96907e4a5979c93c2c25c8c7..3863d7910f2ac45608675f67a242a5f9eac7a0ca 100644
--- a/extensions/browser/guest_view/web_view/web_view_apitest.cc
+++ b/extensions/browser/guest_view/web_view/web_view_apitest.cc
@@ -17,6 +17,7 @@
 #include "extensions/browser/app_window/app_window.h"
 #include "extensions/browser/app_window/app_window_registry.h"
 #include "extensions/browser/extension_host.h"
+#include "extensions/browser/guest_view/extensions_guest_view_manager_delegate.h"
 #include "extensions/browser/guest_view/guest_view_manager.h"
 #include "extensions/browser/guest_view/guest_view_manager_factory.h"
 #include "extensions/browser/guest_view/test_guest_view_manager.h"
@@ -237,9 +238,20 @@ content::WebContents* WebViewAPITest::GetEmbedderWebContents() {
 }
 
 TestGuestViewManager* WebViewAPITest::GetGuestViewManager() {
-  return static_cast<TestGuestViewManager*>(
-      TestGuestViewManager::FromBrowserContext(
-          ShellContentBrowserClient::Get()->GetBrowserContext()));
+  content::BrowserContext* context =
+      ShellContentBrowserClient::Get()->GetBrowserContext();
+  TestGuestViewManager* manager = static_cast<TestGuestViewManager*>(
+      TestGuestViewManager::FromBrowserContext(context));
+  // TestGuestViewManager::WaitForSingleGuestCreated may and will get called
+  // before a guest is created.
+  if (!manager) {
+    manager = static_cast<TestGuestViewManager*>(
+        GuestViewManager::CreateWithDelegate(
+            context,
+            scoped_ptr<guestview::GuestViewManagerDelegate>(
+                new ExtensionsGuestViewManagerDelegate(context))));
+  }
+  return manager;
 }
 
 void WebViewAPITest::SendMessageToGuestAndWait(
@@ -364,6 +376,7 @@ IN_PROC_BROWSER_TEST_F(WebViewAPITest, MAYBE_CloseOnLoadcommit) {
   ExtensionTestMessageListener test_done_listener("done-close-on-loadcommit",
                                                   false);
   ASSERT_TRUE(test_done_listener.WaitUntilSatisfied());
+  fprintf(stderr, ">>%s\n", __PRETTY_FUNCTION__);

[Fady Samuel, 2015/04/23 03:44:52]

Note to self: Delete this.

[Fady Samuel, 2015/04/24 00:11:31]

Done.

 }
 
 // This test verifies that reloading the embedder reloads the guest (and doest