Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1471)

Unified Diff: chrome/browser/first_run/first_run_browsertest.cc

Issue 22560003: Reland "Add option to import from bookmarks html file to ..."" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: does this fix first run browser_tests for bots? Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/first_run/first_run_browsertest.cc
diff --git a/chrome/browser/first_run/first_run_browsertest.cc b/chrome/browser/first_run/first_run_browsertest.cc
index b6fd6fd03430f59e246e4e5f4f4d9ca86b2d030e..62b439b408e8561faf4b64f58931c4f794bf3cf3 100644
--- a/chrome/browser/first_run/first_run_browsertest.cc
+++ b/chrome/browser/first_run/first_run_browsertest.cc
@@ -146,7 +146,7 @@ int MaskExpectedImportState(int expected_import_state) {
// Internet Explorer always exists and always has something to import.
EXPECT_GT(source_profile_count, 0);
#endif
- if (source_profile_count == 0)
+ if (source_profile_count == 1)
gab 2013/09/17 22:06:00 This is wrong, as mentioned in another comment, Au
return expected_import_state & ~first_run::AUTO_IMPORT_PROFILE_IMPORTED;
return expected_import_state;

Powered by Google App Engine
This is Rietveld 408576698