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

Unified Diff: webkit/tools/test_shell/test_shell_mac.mm

Issue 10387010: Select theme resources from ResourceBundle at requested scale factor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge with master. Created 8 years, 7 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: webkit/tools/test_shell/test_shell_mac.mm
diff --git a/webkit/tools/test_shell/test_shell_mac.mm b/webkit/tools/test_shell/test_shell_mac.mm
index 9c38d449d3d8f5e185811ae06ffefc007855f042..d9b6ac2cb5a26140956843c3952d08eead298a84 100644
--- a/webkit/tools/test_shell/test_shell_mac.mm
+++ b/webkit/tools/test_shell/test_shell_mac.mm
@@ -212,7 +212,7 @@ void TestShell::InitializeTestShell(bool layout_test_mode,
// mmap the data pack which holds strings used by WebCore. This is only
// a fatal error if we're bundled, which means we might be running layout
// tests. This is a harmless failure for test_shell_tests.
- g_resource_data_pack = new ui::DataPack(ui::ResourceHandle::kScaleFactor100x);
+ g_resource_data_pack = new ui::DataPack(ui::SCALE_FACTOR_100P);
NSString *resource_path =
[base::mac::FrameworkBundle() pathForResource:@"test_shell"
ofType:@"pak"];
@@ -630,12 +630,9 @@ string16 TestShellWebKitInit::GetLocalizedString(int message_id) {
return msg;
}
-base::StringPiece TestShellWebKitInit::GetDataResource(int resource_id) {
- return base::StringPiece();
-}
-
-base::StringPiece TestShellWebKitInit::GetImageResource(int resource_id,
- float scale_factor) {
+base::StringPiece TestShellWebKitInit::GetDataResource(
+ int resource_id,
+ ui::ScaleFactor scale_factor) {
switch (resource_id) {
case IDR_BROKENIMAGE: {
// Use webkit's broken image icon (16x16)

Powered by Google App Engine
This is Rietveld 408576698