OLD | NEW |
---|---|
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "base/command_line.h" | 5 #include "base/command_line.h" |
6 #include "base/prefs/pref_service.h" | 6 #include "base/prefs/pref_service.h" |
7 #include "base/utf_string_conversions.h" | 7 #include "base/utf_string_conversions.h" |
8 #include "chrome/browser/browser_process.h" | 8 #include "chrome/browser/browser_process.h" |
9 #include "chrome/browser/extensions/component_loader.h" | 9 #include "chrome/browser/extensions/component_loader.h" |
10 #include "chrome/browser/first_run/first_run.h" | 10 #include "chrome/browser/first_run/first_run.h" |
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
64 | 64 |
65 #if !defined(OS_CHROMEOS) | 65 #if !defined(OS_CHROMEOS) |
66 namespace { | 66 namespace { |
67 class FirstRunIntegrationBrowserTest : public InProcessBrowserTest { | 67 class FirstRunIntegrationBrowserTest : public InProcessBrowserTest { |
68 public: | 68 public: |
69 FirstRunIntegrationBrowserTest() {} | 69 FirstRunIntegrationBrowserTest() {} |
70 protected: | 70 protected: |
71 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE { | 71 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE { |
72 InProcessBrowserTest::SetUpCommandLine(command_line); | 72 InProcessBrowserTest::SetUpCommandLine(command_line); |
73 command_line->AppendSwitch(switches::kForceFirstRun); | 73 command_line->AppendSwitch(switches::kForceFirstRun); |
74 EXPECT_FALSE(ProfileManager::DidPerformProfileImport()); | 74 EXPECT_FALSE(first_run::DidPerformProfileImport(NULL)); |
75 | 75 |
76 extensions::ComponentLoader::EnableBackgroundExtensionsForTesting(); | 76 extensions::ComponentLoader::EnableBackgroundExtensionsForTesting(); |
77 | 77 |
78 // The forked import process should run BrowserMain. | 78 // The forked import process should run BrowserMain. |
79 CommandLine import_arguments((CommandLine::NoProgram())); | 79 CommandLine import_arguments((CommandLine::NoProgram())); |
80 import_arguments.AppendSwitch(content::kLaunchAsBrowser); | 80 import_arguments.AppendSwitch(content::kLaunchAsBrowser); |
81 first_run::SetExtraArgumentsForImportProcess(import_arguments); | 81 first_run::SetExtraArgumentsForImportProcess(import_arguments); |
82 } | 82 } |
83 private: | 83 private: |
84 DISALLOW_COPY_AND_ASSIGN(FirstRunIntegrationBrowserTest); | 84 DISALLOW_COPY_AND_ASSIGN(FirstRunIntegrationBrowserTest); |
(...skipping 28 matching lines...) Expand all Loading... | |
113 FirstRunIntegrationBrowserTest::TearDown(); | 113 FirstRunIntegrationBrowserTest::TearDown(); |
114 } | 114 } |
115 | 115 |
116 private: | 116 private: |
117 base::FilePath prefs_file_; | 117 base::FilePath prefs_file_; |
118 | 118 |
119 DISALLOW_COPY_AND_ASSIGN(FirstRunMasterPrefsBrowserTest); | 119 DISALLOW_COPY_AND_ASSIGN(FirstRunMasterPrefsBrowserTest); |
120 }; | 120 }; |
121 } | 121 } |
122 | 122 |
123 IN_PROC_BROWSER_TEST_F(FirstRunIntegrationBrowserTest, WaitForImport) { | 123 // TODO(tapted): Investigate why this fails on Linux bots but does not |
gab
2013/03/07 15:35:49
Perhaps this was the same issue I fixed in http://
gab
2013/03/07 15:53:18
Meh, nvm, this fix was win only.
| |
124 ASSERT_TRUE(ProfileManager::DidPerformProfileImport()); | 124 // reproduce locally. See http://crbug.com/178062 . |
125 #if defined(OS_LINUX) | |
126 #define MAYBE_WaitForImport DISABLED_WaitForImport | |
127 #else | |
128 #define MAYBE_WaitForImport WaitForImport | |
129 #endif | |
130 | |
131 IN_PROC_BROWSER_TEST_F(FirstRunIntegrationBrowserTest, MAYBE_WaitForImport) { | |
132 bool success = false; | |
133 EXPECT_TRUE(first_run::DidPerformProfileImport(&success)); | |
134 // Aura builds skip over the import process. | |
135 #if defined(USE_AURA) | |
136 EXPECT_FALSE(success); | |
137 #else | |
138 EXPECT_TRUE(success); | |
139 #endif | |
125 } | 140 } |
126 | 141 |
127 // Test an import with all import options disabled. This is a regression test | 142 // Test an import with all import options disabled. This is a regression test |
128 // for http://crbug.com/169984 where this would cause the import process to | 143 // for http://crbug.com/169984 where this would cause the import process to |
129 // stay running, and the NTP to be loaded with no apps. | 144 // stay running, and the NTP to be loaded with no apps. |
130 IN_PROC_BROWSER_TEST_F(FirstRunMasterPrefsBrowserTest, | 145 IN_PROC_BROWSER_TEST_F(FirstRunMasterPrefsBrowserTest, |
131 ImportNothingAndShowNewTabPage) { | 146 ImportNothingAndShowNewTabPage) { |
132 ASSERT_TRUE(ProfileManager::DidPerformProfileImport()); | 147 EXPECT_TRUE(first_run::DidPerformProfileImport(NULL)); |
133 ui_test_utils::NavigateToURLWithDisposition( | 148 ui_test_utils::NavigateToURLWithDisposition( |
134 browser(), GURL(chrome::kChromeUINewTabURL), CURRENT_TAB, | 149 browser(), GURL(chrome::kChromeUINewTabURL), CURRENT_TAB, |
135 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION); | 150 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION); |
136 content::WebContents* tab = browser()->tab_strip_model()->GetWebContentsAt(0); | 151 content::WebContents* tab = browser()->tab_strip_model()->GetWebContentsAt(0); |
137 EXPECT_EQ(1, tab->GetMaxPageID()); | 152 EXPECT_EQ(1, tab->GetMaxPageID()); |
138 } | 153 } |
139 | 154 |
140 #endif // !defined(OS_CHROMEOS) | 155 #endif // !defined(OS_CHROMEOS) |
OLD | NEW |