X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Fchrome%2Fbrowser%2Fextensions%2Fisolated_app_browsertest.cc;h=fc70dad9822817639979b25b56c73edb3dbbfac4;hb=4a1a0bdd01eef90b0826a0e761d3379d3715c10f;hp=26187c2c981bd5e9025667a6d13db4e81ac1ade6;hpb=b1be5ca53587d23e7aeb77b26861fdc0a181ffd8;p=platform%2Fframework%2Fweb%2Fcrosswalk.git

diff --git a/src/chrome/browser/extensions/isolated_app_browsertest.cc b/src/chrome/browser/extensions/isolated_app_browsertest.cc
index 26187c2..fc70dad 100644
--- a/src/chrome/browser/extensions/isolated_app_browsertest.cc
+++ b/src/chrome/browser/extensions/isolated_app_browsertest.cc
@@ -5,8 +5,6 @@
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_apitest.h"
-#include "chrome/browser/extensions/extension_service.h"
-#include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
@@ -18,6 +16,7 @@
 #include "content/public/browser/web_contents.h"
 #include "content/public/test/browser_test_utils.h"
 #include "extensions/browser/extension_host.h"
+#include "extensions/browser/extension_registry.h"
 #include "extensions/browser/process_map.h"
 #include "extensions/common/switches.h"
 #include "net/dns/mock_host_resolver.h"
@@ -28,8 +27,10 @@
 using content::ExecuteScript;
 using content::ExecuteScriptAndExtractString;
 using content::NavigationController;
-using content::WebContents;
 using content::RenderViewHost;
+using content::WebContents;
+
+namespace extensions {
 
 namespace {
 
@@ -107,35 +108,29 @@ class IsolatedAppTest : public ExtensionBrowserTest {
     return actual_cookie.find(cookie) != std::string::npos;
   }
 
-  const extensions::Extension* GetInstalledApp(WebContents* contents) {
-    const extensions::Extension* installed_app = NULL;
-    Profile* profile =
-        Profile::FromBrowserContext(contents->GetBrowserContext());
-    ExtensionService* service = profile->GetExtensionService();
-    if (service) {
-      std::set<std::string> extension_ids =
-          extensions::ProcessMap::Get(profile)->GetExtensionsInProcess(
-              contents->GetRenderViewHost()->GetProcess()->GetID());
-      for (std::set<std::string>::iterator iter = extension_ids.begin();
-           iter != extension_ids.end(); ++iter) {
-        installed_app = service->extensions()->GetByID(*iter);
-        if (installed_app && installed_app->is_app())
-          return installed_app;
-      }
+  const Extension* GetInstalledApp(WebContents* contents) {
+    content::BrowserContext* browser_context = contents->GetBrowserContext();
+    ExtensionRegistry* registry = ExtensionRegistry::Get(browser_context);
+    std::set<std::string> extension_ids =
+        ProcessMap::Get(browser_context)->GetExtensionsInProcess(
+            contents->GetRenderViewHost()->GetProcess()->GetID());
+    for (std::set<std::string>::iterator iter = extension_ids.begin();
+         iter != extension_ids.end(); ++iter) {
+      const Extension* installed_app =
+          registry->enabled_extensions().GetByID(*iter);
+      if (installed_app && installed_app->is_app())
+        return installed_app;
     }
     return NULL;
   }
 
  private:
-  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+  virtual void SetUpCommandLine(base::CommandLine* command_line) OVERRIDE {
     ExtensionBrowserTest::SetUpCommandLine(command_line);
-    command_line->AppendSwitch(
-        extensions::switches::kEnableExperimentalExtensionApis);
+    command_line->AppendSwitch(switches::kEnableExperimentalExtensionApis);
   }
 };
 
-}  // namespace
-
 IN_PROC_BROWSER_TEST_F(IsolatedAppTest, CrossProcessClientRedirect) {
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
@@ -304,7 +299,6 @@ IN_PROC_BROWSER_TEST_F(IsolatedAppTest, MAYBE_CookieIsolation) {
   EXPECT_TRUE(HasCookie(tab0, "app1=3"));
   EXPECT_FALSE(HasCookie(tab0, "app2"));
   EXPECT_FALSE(HasCookie(tab0, "normalPage"));
-
 }
 
 // This test is disabled due to being flaky. http://crbug.com/145588
@@ -379,13 +373,12 @@ IN_PROC_BROWSER_TEST_F(IsolatedAppTest, DISABLED_NoCookieIsolationWithoutApp) {
   EXPECT_EQ("ls_normal", result);
 }
 
-// Test timing out on Windows debug bots.
 // http://crbug.com/174926
-#if defined(OS_WIN) && !defined(NDEBUG)
+#if (defined(OS_WIN) && !defined(NDEBUG)) || defined(OS_MACOSX)
 #define MAYBE_SubresourceCookieIsolation DISABLED_SubresourceCookieIsolation
 #else
 #define MAYBE_SubresourceCookieIsolation SubresourceCookieIsolation
-#endif  // defined(OS_WIN) && !defined(NDEBUG)
+#endif  // (defined(OS_WIN) && !defined(NDEBUG)) || defined(OS_MACOSX)
 
 // Tests that subresource and media requests use the app's cookie store.
 // See http://crbug.com/141172.
@@ -595,3 +588,7 @@ IN_PROC_BROWSER_TEST_F(IsolatedAppTest, DISABLED_SessionStorage) {
       kRetrieveSessionStorage.c_str(), &result));
   EXPECT_EQ("ss_normal", result);
 }
+
+}  // namespace
+
+}  // namespace extensions