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

Side by Side Diff: chrome/browser/ui/prefs/prefs_tab_helper_browsertest.cc

Issue 10107014: Migrate WebKit "global script" font prefs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: compile fixes Created 8 years, 8 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/file_util.h" 5 #include "base/file_util.h"
6 #include "base/path_service.h" 6 #include "base/path_service.h"
7 #include "chrome/browser/prefs/pref_service.h" 7 #include "chrome/browser/prefs/pref_service.h"
8 #include "chrome/browser/profiles/profile.h" 8 #include "chrome/browser/profiles/profile.h"
9 #include "chrome/browser/ui/browser.h" 9 #include "chrome/browser/ui/browser.h"
10 #include "chrome/common/chrome_constants.h" 10 #include "chrome/common/chrome_constants.h"
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
45 #if defined(OS_WIN) 45 #if defined(OS_WIN)
46 // Make the copy writable. On POSIX we assume the umask allows files 46 // Make the copy writable. On POSIX we assume the umask allows files
47 // we create to be writable. 47 // we create to be writable.
48 if (!::SetFileAttributesW(default_pref_file.value().c_str(), 48 if (!::SetFileAttributesW(default_pref_file.value().c_str(),
49 FILE_ATTRIBUTE_NORMAL)) return false; 49 FILE_ATTRIBUTE_NORMAL)) return false;
50 #endif 50 #endif
51 return true; 51 return true;
52 } 52 }
53 }; 53 };
54 54
55 // This tests migration like:
56 // webkit.webprefs.default_charset -> webkit.webprefs.global.default_charset
57 // This was needed for per-tab prefs, which have since been removed. So
58 // this will soon be replaced with the reverse migration.
55 IN_PROC_BROWSER_TEST_F(PrefsTabHelperBrowserTest, NonGlobalPrefsAreMigrated) { 59 IN_PROC_BROWSER_TEST_F(PrefsTabHelperBrowserTest, NonGlobalPrefsAreMigrated) {
56 PrefService* prefs = browser()->profile()->GetPrefs(); 60 PrefService* prefs = browser()->profile()->GetPrefs();
57 61
58 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kDefaultCharset)); 62 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kDefaultCharset));
59 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitDefaultFontSize)); 63 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitDefaultFontSize));
60 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitDefaultFixedFontSize)); 64 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitDefaultFixedFontSize));
61 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitMinimumFontSize)); 65 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitMinimumFontSize));
62 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitMinimumLogicalFontSize)); 66 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitMinimumLogicalFontSize));
63 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitCursiveFontFamily));
64 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitFantasyFontFamily));
65 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitFixedFontFamily));
66 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitSansSerifFontFamily));
67 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitSerifFontFamily));
68 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitStandardFontFamily));
69 67
70 EXPECT_EQ("ISO-8859-1", prefs->GetString(prefs::kGlobalDefaultCharset)); 68 EXPECT_EQ("ISO-8859-1", prefs->GetString(prefs::kGlobalDefaultCharset));
71 EXPECT_EQ(42, prefs->GetInteger(prefs::kWebKitGlobalDefaultFontSize)); 69 EXPECT_EQ(42, prefs->GetInteger(prefs::kWebKitGlobalDefaultFontSize));
72 EXPECT_EQ(42, 70 EXPECT_EQ(42,
73 prefs->GetInteger(prefs::kWebKitGlobalDefaultFixedFontSize)); 71 prefs->GetInteger(prefs::kWebKitGlobalDefaultFixedFontSize));
74 EXPECT_EQ(42, prefs->GetInteger(prefs::kWebKitGlobalMinimumFontSize)); 72 EXPECT_EQ(42, prefs->GetInteger(prefs::kWebKitGlobalMinimumFontSize));
75 EXPECT_EQ(42, 73 EXPECT_EQ(42,
76 prefs->GetInteger(prefs::kWebKitGlobalMinimumLogicalFontSize)); 74 prefs->GetInteger(prefs::kWebKitGlobalMinimumLogicalFontSize));
75 };
76
77 // This tests migration like:
78 // webkit.webprefs.standard_font_family -> webkit.webprefs.fonts.standard.Zyyy
79 IN_PROC_BROWSER_TEST_F(PrefsTabHelperBrowserTest, PrefsAreMigratedToFontMap) {
80 PrefService* prefs = browser()->profile()->GetPrefs();
81
82 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitOldCursiveFontFamily));
83 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitOldFantasyFontFamily));
84 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitOldFixedFontFamily));
85 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitOldSansSerifFontFamily));
86 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitOldSerifFontFamily));
87 EXPECT_EQ(NULL, prefs->FindPreference(prefs::kWebKitOldStandardFontFamily));
77 EXPECT_EQ("CursiveFontFamily", 88 EXPECT_EQ("CursiveFontFamily",
78 prefs->GetString(prefs::kWebKitGlobalCursiveFontFamily)); 89 prefs->GetString(prefs::kWebKitCursiveFontFamily));
79 EXPECT_EQ("FantasyFontFamily", 90 EXPECT_EQ("FantasyFontFamily",
80 prefs->GetString(prefs::kWebKitGlobalFantasyFontFamily)); 91 prefs->GetString(prefs::kWebKitFantasyFontFamily));
81 EXPECT_EQ("FixedFontFamily", 92 EXPECT_EQ("FixedFontFamily",
82 prefs->GetString(prefs::kWebKitGlobalFixedFontFamily)); 93 prefs->GetString(prefs::kWebKitFixedFontFamily));
83 EXPECT_EQ("SansSerifFontFamily", 94 EXPECT_EQ("SansSerifFontFamily",
84 prefs->GetString(prefs::kWebKitGlobalSansSerifFontFamily)); 95 prefs->GetString(prefs::kWebKitSansSerifFontFamily));
85 EXPECT_EQ("SerifFontFamily", 96 EXPECT_EQ("SerifFontFamily",
86 prefs->GetString(prefs::kWebKitGlobalSerifFontFamily)); 97 prefs->GetString(prefs::kWebKitSerifFontFamily));
87 EXPECT_EQ("StandardFontFamily", 98 EXPECT_EQ("StandardFontFamily",
88 prefs->GetString(prefs::kWebKitGlobalStandardFontFamily)); 99 prefs->GetString(prefs::kWebKitStandardFontFamily));
89 } 100 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698