Implement chromium.self in content scripts, fix bugs

chrome/browser/extensions/extension_view_unittest.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "base/message_loop.h"
 #include "base/ref_counted.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/renderer_host/render_view_host.h"
-#include "chrome/browser/profile.h"
 #include "chrome/browser/extensions/extension_error_reporter.h"
 #include "chrome/browser/extensions/extension_view.h"
 #include "chrome/browser/extensions/extensions_service.h"
+#include "chrome/browser/extensions/test_extension_loader.h"
 #include "chrome/common/chrome_paths.h"
-#include "chrome/common/notification_service.h"
 #include "chrome/test/in_process_browser_test.h"
 #include "chrome/test/ui_test_utils.h"
 
 namespace {
 
 // How long to wait for the extension to put up a javascript alert before giving
 // up.
 const int kAlertTimeoutMs = 20000;
 
-// How long to wait for the extension to load before giving up.
-const int kLoadTimeoutMs = 10000;
-
 // The extension we're using as our test case.
 const char* kExtensionId = "00123456789abcdef0123456789abcdef0123456";
 
+};  // namespace
+
 // This class starts up an extension process and waits until it tries to put
 // up a javascript alert.
 class MockExtensionView : public ExtensionView {
  public:
   MockExtensionView(Extension* extension, const GURL& url, Profile* profile)
       : ExtensionView(extension, url, profile), got_message_(false) {
     InitHidden();
     MessageLoop::current()->PostDelayedTask(FROM_HERE,
         new MessageLoop::QuitTask, kAlertTimeoutMs);
     ui_test_utils::RunMessageLoop();
@@ skipping 13 matching lines @@
 
     // Call super, otherwise we'll leak reply_msg.
     ExtensionView::RunJavaScriptMessage(
         message, default_prompt, frame_url, flags,
         reply_msg, did_suppress_message);
   }
 
   bool got_message_;
 };
 
-// This class waits for a specific extension to be loaded.
-class ExtensionLoadedObserver : public NotificationObserver {
- public:
-  explicit ExtensionLoadedObserver() : extension_(NULL)  {
-    registrar_.Add(this, NotificationType::EXTENSIONS_LOADED,
-        NotificationService::AllSources());
-  }
-
-  Extension* WaitForExtension() {
-    MessageLoop::current()->PostDelayedTask(FROM_HERE,
-        new MessageLoop::QuitTask, kLoadTimeoutMs);
-    ui_test_utils::RunMessageLoop();
-    return extension_;
-  }
-
- private:
-  virtual void Observe(NotificationType type, const NotificationSource& source,
-                       const NotificationDetails& details) {
-    if (type == NotificationType::EXTENSIONS_LOADED) {
-      ExtensionList* extensions = Details<ExtensionList>(details).ptr();
-      for (size_t i = 0; i < (*extensions).size(); i++) {
-        if ((*extensions)[i]->id() == kExtensionId) {
-          extension_ = (*extensions)[i];
-          MessageLoopForUI::current()->Quit();
-          break;
-        }
-      }
-    } else {
-      NOTREACHED();
-    }
-  }
-
-  NotificationRegistrar registrar_;
-  Extension* extension_;
-};
-
-}  // namespace
-
 class ExtensionViewTest : public InProcessBrowserTest {
  public:
   virtual void SetUp() {
     // Initialize the error reporter here, otherwise BrowserMain will create it
     // with the wrong MessageLoop.
     ExtensionErrorReporter::Init(false);
 
     // Use single-process in an attempt to speed it up and make it less flaky.
     //EnableSingleProcess();
 
     InProcessBrowserTest::SetUp();
   }
 };
 
 // Tests that ExtensionView starts an extension process and runs the script
 // contained in the extension's "index.html" file.
 IN_PROC_BROWSER_TEST_F(ExtensionViewTest, Index) {
-  // Create an observer first to be sure we have the notification registered
-  // before it's sent.
-  ExtensionLoadedObserver observer;
-
   // Get the path to our extension.
   FilePath path;
   ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &path));
   path = path.AppendASCII("extensions").
       AppendASCII("good").AppendASCII("extension1").AppendASCII("1");
   ASSERT_TRUE(file_util::DirectoryExists(path));  // sanity check
 
-  // Load it.
-  Profile* profile = browser()->profile();
-  profile->GetExtensionsService()->Init();
-  profile->GetExtensionsService()->LoadExtension(path);
-
-  // Now wait for it to load, and grab a pointer to it.
-  Extension* extension = observer.WaitForExtension();
+  // Wait for the extension to load and grab a pointer to it.
+  TestExtensionLoader loader(browser()->profile());
+  Extension* extension = loader.Load(kExtensionId, path);
   ASSERT_TRUE(extension);
   GURL url = Extension::GetResourceURL(extension->url(), "toolstrip1.html");
 
   // Start the extension process and wait for it to show a javascript alert.
-  MockExtensionView view(extension, url, profile);
+  MockExtensionView view(extension, url, browser()->profile());
   EXPECT_TRUE(view.got_message());
 }