ui: Remove ResourceBundle::LoadCommonResources().

ui/base/resource/resource_bundle_mac.mm

 // 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"
 
 #import <AppKit/AppKit.h>
 
 #include "base/basictypes.h"
 #include "base/file_util.h"
@@ skipping 30 matching lines @@
   if (!resource_path) {
     // Return just the name of the pack file.
     return base::FilePath(base::SysNSStringToUTF8(name) + ".pak");
   }
 
   return base::FilePath([resource_path fileSystemRepresentation]);
 }
 
 }  // namespace
 
-void ResourceBundle::LoadCommonResources() {
-  AddDataPackFromPath(GetResourcesPakFilePath(@"chrome_100_percent",
-                        nil), SCALE_FACTOR_100P);
-  AddDataPackFromPath(GetResourcesPakFilePath(@"webkit_resources_100_percent",
-                        nil), SCALE_FACTOR_100P);

[tony, 2014/08/07 16:29:34]

Have you verified on OSX we don't need to load webkit_resources_100_percent? 
Maybe remove that in a separate patch?

[oshima, 2014/08/07 17:01:10]

yes, please check if this is needed. If this is necessary, then InitParams needs
to take two files for each scale (which I'd like to avoid)

[tfarina, 2014/08/07 17:12:33]

The only way I can check is with trybots, as I don't have a Mac to try locally
myself.

-
-  // On Mac we load 1x and 2x resources and we let the UI framework decide
-  // which one to use.
-  if (IsScaleFactorSupported(SCALE_FACTOR_200P)) {
-    AddDataPackFromPath(GetResourcesPakFilePath(@"chrome_200_percent",
-                          nil), SCALE_FACTOR_200P);
-    AddDataPackFromPath(GetResourcesPakFilePath(@"webkit_resources_200_percent",
-                          nil), SCALE_FACTOR_200P);
-  }
-}
-
 base::FilePath ResourceBundle::GetLocaleFilePath(const std::string& app_locale,
                                                  bool test_file_exists) {
   NSString* mac_locale = base::SysUTF8ToNSString(app_locale);
 
   // Mac OS X uses "_" instead of "-", so swap to get a Mac-style value.
   mac_locale = [mac_locale stringByReplacingOccurrencesOfString:@"-"
                                                      withString:@"_"];
 
   // On disk, the "en_US" resources are just "en" (http://crbug.com/25578).
   if ([mac_locale isEqual:@"en_US"])
@@ skipping 71 matching lines @@
 
   // Another thread raced the load and has already cached the image.
   if (images_.count(resource_id))
     return images_[resource_id];
 
   images_[resource_id] = image;
   return images_[resource_id];
 }
 
 }  // namespace ui