Load the resources for max scale factor first.

ui/base/resource/resource_bundle_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/base/resource/resource_bundle.h"
 
 #include "base/base_paths.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/memory/ref_counted_memory.h"
@@ skipping 229 matching lines @@
   const gfx::Font* font =
       &resource_bundle.GetFont(ui::ResourceBundle::BaseFont);
   EXPECT_TRUE(font);
 }
 
 TEST(ResourceBundle, LoadDataResourceBytes) {
   // On Windows, the default data is compiled into the binary so this does
   // nothing.
   ScopedTempDir dir;
   ASSERT_TRUE(dir.CreateUniqueTempDir());
-  FilePath data_path = dir.path().Append(FILE_PATH_LITERAL("sample.pak"));
+  FilePath data_path = dir.path().AppendASCII("sample.pak");
 
   // Put the ResourceBundle in a different scope so that it's destroyed before
   // the ScopedTempDir.
   {
     // Verify that we don't crash when trying to load a resource that is not
     // found.  In some cases, we fail to mmap resources.pak, but try to keep
     // going anyway.
     ResourceBundle resource_bundle(NULL);
 
     // Dump contents into the pak file.
@@ skipping 16 matching lines @@
         kUnfoundResourceId));
   }
 }
 
 TEST(ResourceBundle, GetRawDataResource) {
 
   // On Windows, the default data is compiled into the binary so this does
   // nothing.
   ScopedTempDir dir;
   ASSERT_TRUE(dir.CreateUniqueTempDir());
-  FilePath locale_path = dir.path().Append(FILE_PATH_LITERAL("empty.pak"));
-  FilePath data_path = dir.path().Append(FILE_PATH_LITERAL("sample.pak"));
-  FilePath data_2x_path = dir.path().Append(FILE_PATH_LITERAL("sample_2x.pak"));
+  FilePath locale_path = dir.path().AppendASCII("empty.pak");
+  FilePath data_path = dir.path().AppendASCII("sample.pak");
+  FilePath data_2x_path = dir.path().AppendASCII("sample_2x.pak");
 
   {
     ResourceBundle resource_bundle(NULL);
     // Dump contents into the pak files.
     ASSERT_EQ(file_util::WriteFile(locale_path, kEmptyPakContents,
         kEmptyPakSize), static_cast<int>(kEmptyPakSize));
     ASSERT_EQ(file_util::WriteFile(data_path, kSamplePakContents,
         kSamplePakSize), static_cast<int>(kSamplePakSize));
     ASSERT_EQ(file_util::WriteFile(data_2x_path, kSamplePakContents2x,
         kSamplePakSize2x), static_cast<int>(kSamplePakSize2x));
@@ skipping 27 matching lines @@
 }
 
 // Test requesting image reps at various scale factors from the image returned
 // via ResourceBundle::GetImageNamed().
 TEST(ResourceBundle, GetImageNamed) {
   // On Windows, the default data is compiled into the binary so this does
   // nothing.
   ScopedTempDir dir;
   ASSERT_TRUE(dir.CreateUniqueTempDir());
 
-  FilePath locale_path = dir.path().Append(FILE_PATH_LITERAL("empty.pak"));
-  FilePath data_path = dir.path().Append(FILE_PATH_LITERAL("sample.pak"));
-  FilePath data_2x_path = dir.path().Append(FILE_PATH_LITERAL("sample_2x.pak"));
+  FilePath locale_path = dir.path().AppendASCII("empty.pak");
+  FilePath data_default_path = dir.path().AppendASCII("sample.pak");
+  FilePath data_1x_path = dir.path().AppendASCII("sample_1x.pak");
+  FilePath data_2x_path = dir.path().AppendASCII("sample_2x.pak");
 
-  {
-    // Create the pak files.
-    ASSERT_EQ(file_util::WriteFile(locale_path, kEmptyPakContents,
-        kEmptyPakSize), static_cast<int>(kEmptyPakSize));
-    CreateDataPackWithSingleBitmap(data_path, 10);
-    CreateDataPackWithSingleBitmap(data_2x_path, 20);
+  // Create the pak files.
+  ASSERT_EQ(file_util::WriteFile(locale_path, kEmptyPakContents,
+      kEmptyPakSize), static_cast<int>(kEmptyPakSize));
+  CreateDataPackWithSingleBitmap(data_default_path, 10);
+  CreateDataPackWithSingleBitmap(data_1x_path, 10);
+  CreateDataPackWithSingleBitmap(data_2x_path, 20);
 
-    // Load the regular and 2x pak files.
-    ResourceBundle resource_bundle(NULL);
-    resource_bundle.LoadTestResources(data_path, locale_path);
-    resource_bundle.AddDataPackFromPath(data_2x_path, SCALE_FACTOR_200P);
+  // Load the regular and 2x pak files.
+  ResourceBundle resource_bundle(NULL);
+  resource_bundle.LoadTestResources(data_default_path, locale_path);
+  resource_bundle.AddDataPackFromPath(data_1x_path, SCALE_FACTOR_100P);
+  resource_bundle.AddDataPackFromPath(data_2x_path, SCALE_FACTOR_200P);
 
-    gfx::ImageSkia* image_skia = resource_bundle.GetImageSkiaNamed(3);
+  gfx::ImageSkia* image_skia = resource_bundle.GetImageSkiaNamed(3);
 
-    // Resource ID 3 exists in both 1x and 2x paks. Image reps should be
-    // available for both scale factors in |image_skia|.
-    gfx::ImageSkiaRep image_rep =
-        image_skia->GetRepresentation(ui::SCALE_FACTOR_100P);
-    EXPECT_EQ(ui::SCALE_FACTOR_100P, image_rep.scale_factor());
-    image_rep = image_skia->GetRepresentation(ui::SCALE_FACTOR_200P);
-    EXPECT_EQ(ui::SCALE_FACTOR_200P, image_rep.scale_factor());
+  EXPECT_EQ(ui::GetMaxScaleFactor(),
+            image_skia->image_reps()[0].scale_factor());

[pkotwicz, 2012/11/07 06:40:44]

Shouldn't you call ui::test::SetSupportedScaleFactos() so that this works on
retina mac?
This may be out of date, but it seems as if ui::GetMaxScaleFactor() checks for
the maximum supported scale factor on non Chromeos.

[oshima, 2012/11/07 22:35:30]

Current code is a bit broken as we included 200P even if the device
doesn't support it. I fixed it so that SetSupportedScaleFactor works for all
platforms, and added explicit 200P.

 
-    // The 1.4x pack was not loaded. Requesting the 1.4x resource should return
-    // either the 1x or the 2x resource.
-    image_rep = image_skia->GetRepresentation(ui::SCALE_FACTOR_140P);
-    EXPECT_TRUE(image_rep.scale_factor() == ui::SCALE_FACTOR_100P ||
-                image_rep.scale_factor() == ui::SCALE_FACTOR_200P);
-  }
+  // Resource ID 3 exists in both 1x and 2x paks. Image reps should be
+  // available for both scale factors in |image_skia|.
+  gfx::ImageSkiaRep image_rep =
+      image_skia->GetRepresentation(ui::SCALE_FACTOR_100P);
+  EXPECT_EQ(ui::SCALE_FACTOR_100P, image_rep.scale_factor());
+  image_rep = image_skia->GetRepresentation(ui::SCALE_FACTOR_200P);
+  EXPECT_EQ(ui::SCALE_FACTOR_200P, image_rep.scale_factor());
+
+  // The 1.4x pack was not loaded. Requesting the 1.4x resource should return
+  // either the 1x or the 2x resource.
+  image_rep = image_skia->GetRepresentation(ui::SCALE_FACTOR_140P);
+  EXPECT_TRUE(image_rep.scale_factor() == ui::SCALE_FACTOR_100P ||
+              image_rep.scale_factor() == ui::SCALE_FACTOR_200P);
+}
+
+TEST(ResourceBundle, FallbackToNone) {
+  // On Windows, the default data is compiled into the binary so this does
+  // nothing.
+  ScopedTempDir dir;
+  ASSERT_TRUE(dir.CreateUniqueTempDir());
+
+  FilePath locale_path = dir.path().AppendASCII("empty.pak");
+  FilePath data_default_path = dir.path().AppendASCII("sample.pak");
+
+  // Create the pak files.
+  ASSERT_EQ(file_util::WriteFile(locale_path, kEmptyPakContents,
+      kEmptyPakSize), static_cast<int>(kEmptyPakSize));
+  CreateDataPackWithSingleBitmap(data_default_path, 10);
+
+    // Load the regular pak files only.
+  ResourceBundle resource_bundle(NULL);
+  resource_bundle.LoadTestResources(data_default_path, locale_path);
+
+  gfx::ImageSkia* image_skia = resource_bundle.GetImageSkiaNamed(3);
+  EXPECT_EQ(1u, image_skia->image_reps().size());
+  EXPECT_EQ(ui::SCALE_FACTOR_100P,
+            image_skia->image_reps()[0].scale_factor());
 }
 
 }  // namespace ui