Reland "cros - Update appearance of resolution change notification."

This is a reland of f57e1c3 which was reverted due to being blamed for the msan failures which were fixed by d8f72fd Original change's description: > cros - Update appearance of resolution change notification. > > This removes the last raster asset in ash_resources.grd, so remove that > file and the associated build target. > > Bug: 840509,505953 > Change-Id: I43ec69b5b8b9fc34aa71a1b65f204447e73b133c > Reviewed-on: https://chromium-review.googlesource.com/1050523 > Commit-Queue: Evan Stade <estade@chromium.org> > Reviewed-by: James Cook <jamescook@chromium.org> > Reviewed-by: Scott Violet <sky@chromium.org> > Cr-Commit-Position: refs/heads/master@{#557292} TBR=jamescook@chromium.org,sky@chromium.org Bug: 840509, 505953 Change-Id: Iae0b07cbf2e14e307b74e11d07d21290ee72291b Reviewed-on: https://chromium-review.googlesource.com/1054222 Commit-Queue: Evan Stade <estade@chromium.org> Reviewed-by: Scott Violet <sky@chromium.org> Reviewed-by: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#557679}
hylschwy · May 10, 2018 · 13e8011 · 13e8011
1 parent f7e9d43
commit 13e8011
Show file tree

Hide file tree

Showing 23 changed files with 20 additions and 80 deletions.
diff --git a/ash/BUILD.gn b/ash/BUILD.gn
@@ -1208,7 +1208,6 @@ component("ash") {
   public_deps = [
     "//ash/public/cpp",
     "//ash/public/cpp/vector_icons",
-    "//ash/resources",
     "//ash/resources/vector_icons",
     "//ash/strings",
     "//ash/wayland",
@@ -1831,7 +1830,6 @@ test("ash_unittests") {
     "//ash/public/cpp",
     "//ash/public/cpp:unit_tests",
     "//ash/public/cpp/vector_icons",
-    "//ash/resources",
     "//ash/resources/vector_icons",
     "//ash/strings",
     "//ash/touch_hud",
@@ -2130,7 +2128,6 @@ static_library("test_support_common") {
     "//ash/components/fast_ink",
     "//ash/public/cpp",
     "//ash/public/interfaces:test_interfaces",
-    "//ash/resources",
     "//base",
     "//base:i18n",
     "//base/test:test_support",
@@ -2232,7 +2229,6 @@ repack("ash_service_resources") {
   output = "$root_out_dir/ash_service_resources.pak"
   sources = [
     "$root_gen_dir/ash/components/resources/ash_components_resources_100_percent.pak",
-    "$root_gen_dir/ash/resources/ash_resources_100_percent.pak",
     "$root_gen_dir/ash/strings/ash_strings_en-US.pak",
     "$root_gen_dir/ui/chromeos/resources/ui_chromeos_resources_100_percent.pak",
     "$root_gen_dir/ui/chromeos/strings/ui_chromeos_strings_en-US.pak",
@@ -2243,7 +2239,6 @@ repack("ash_service_resources") {
   ]
   deps = [
     "//ash/components/resources",
-    "//ash/resources",
     "//ash/strings",
     "//ui/chromeos/resources",
     "//ui/chromeos/strings",
@@ -2258,14 +2253,12 @@ repack("ash_service_resources_200") {
   output = "$root_out_dir/ash_service_resources_200.pak"
   sources = [
     "$root_gen_dir/ash/components/resources/ash_components_resources_200_percent.pak",
-    "$root_gen_dir/ash/resources/ash_resources_200_percent.pak",
     "$root_gen_dir/ui/chromeos/resources/ui_chromeos_resources_200_percent.pak",
     "$root_gen_dir/ui/resources/ui_resources_200_percent.pak",
     "$root_gen_dir/ui/views/resources/views_resources_200_percent.pak",
   ]
   deps = [
     "//ash/components/resources",
-    "//ash/resources",
     "//ui/chromeos/resources",
     "//ui/resources",
     "//ui/views/resources",

diff --git a/ash/display/resolution_notification_controller.cc b/ash/display/resolution_notification_controller.cc
@@ -6,15 +6,14 @@
 
 #include <utility>
 
-#include "ash/resources/grit/ash_resources.h"
+#include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/session/session_controller.h"
 #include "ash/shell.h"
 #include "ash/strings/grit/ash_strings.h"
 #include "ash/system/screen_layout_observer.h"
 #include "base/strings/utf_string_conversions.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/l10n/time_format.h"
-#include "ui/base/resource/resource_bundle.h"
 #include "ui/display/display.h"
 #include "ui/display/manager/display_manager.h"
 #include "ui/display/manager/managed_display_info.h"
@@ -221,17 +220,22 @@ void ResolutionNotificationController::CreateOrUpdateNotification(
                 base::UTF8ToUTF16(
                     change_info_->current_resolution.size().ToString()));
 
-  ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
-  auto notification = std::make_unique<Notification>(
-      message_center::NOTIFICATION_TYPE_SIMPLE, kNotificationId, message,
-      timeout_message, bundle.GetImageNamed(IDR_AURA_NOTIFICATION_DISPLAY),
-      base::string16() /* display_source */, GURL(),
-      message_center::NotifierId(message_center::NotifierId::SYSTEM_COMPONENT,
-                                 kNotifierDisplayResolutionChange),
-      data,
-      base::MakeRefCounted<message_center::ThunkNotificationDelegate>(
-          weak_factory_.GetWeakPtr()));
-  notification->SetSystemPriority();
+  std::unique_ptr<Notification> notification =
+      Notification::CreateSystemNotification(
+          message_center::NOTIFICATION_TYPE_SIMPLE, kNotificationId, message,
+          timeout_message, gfx::Image(),
+          base::string16(),  // display_source
+          GURL(),
+          message_center::NotifierId(
+              message_center::NotifierId::SYSTEM_COMPONENT,
+              kNotifierDisplayResolutionChange),
+          data,
+          base::MakeRefCounted<message_center::ThunkNotificationDelegate>(
+              weak_factory_.GetWeakPtr()),
+          kNotificationScreenIcon,
+          message_center::SystemNotificationWarningLevel::NORMAL);
+  notification->set_priority(message_center::SYSTEM_PRIORITY);
+
   message_center->AddNotification(std::move(notification));
 }
 

diff --git a/ash/resources/BUILD.gn b/ash/resources/BUILD.gn
@@ -9,15 +9,6 @@ import("//ui/base/ui_features.gni")
 assert(is_chromeos)
 assert(enable_hidpi)
 
-grit("resources") {
-  source = "ash_resources.grd"
-  outputs = [
-    "grit/ash_resources.h",
-    "ash_resources_100_percent.pak",
-    "ash_resources_200_percent.pak",
-  ]
-}
-
 # Repacks resources needed for ash_unittests, etc. at a given scale.
 # TODO(msw): Use ui_test.pak instead of its pieces? (no 200% support?)
 template("ash_test_resources") {
@@ -29,7 +20,6 @@ template("ash_test_resources") {
     sources = [
       "$root_gen_dir/ash/components/resources/ash_components_resources_${percent}_percent.pak",
       "$root_gen_dir/ash/public/cpp/resources/ash_public_unscaled_resources.pak",
-      "$root_gen_dir/ash/resources/ash_resources_${percent}_percent.pak",
       "$root_gen_dir/ui/app_list/resources/app_list_resources_${percent}_percent.pak",
       "$root_gen_dir/ui/chromeos/resources/ui_chromeos_resources_${percent}_percent.pak",
       "$root_gen_dir/ui/resources/ui_resources_${percent}_percent.pak",
@@ -50,7 +40,6 @@ template("ash_test_resources") {
     deps = [
       "//ash/components/resources",
       "//ash/public/cpp/resources:ash_public_unscaled_resources",
-      "//ash/resources",
       "//mojo/public/js:resources",
       "//ui/app_list/resources",
       "//ui/chromeos/resources",

diff --git a/ash/resources/ash_resources.grd b/ash/resources/ash_resources.grd
diff --git a/ash/resources/default_100_percent/cros/notification/display_notification_icon.png b/ash/resources/default_100_percent/cros/notification/display_notification_icon.png
diff --git a/ash/resources/default_200_percent/cros/notification/display_notification_icon.png b/ash/resources/default_200_percent/cros/notification/display_notification_icon.png
diff --git a/ash/system/bluetooth/bluetooth_notification_controller.cc b/ash/system/bluetooth/bluetooth_notification_controller.cc
@@ -7,7 +7,6 @@
 #include <memory>
 #include <utility>
 
-#include "ash/resources/grit/ash_resources.h"
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/strings/grit/ash_strings.h"
 #include "base/bind.h"

diff --git a/ash/system/locale/locale_notification_controller.cc b/ash/system/locale/locale_notification_controller.cc
@@ -8,7 +8,6 @@
 #include <utility>
 
 #include "ash/public/cpp/vector_icons/vector_icons.h"
-#include "ash/resources/grit/ash_resources.h"
 #include "ash/strings/grit/ash_strings.h"
 #include "ash/system/tray/system_tray_notifier.h"
 #include "base/strings/string16.h"

diff --git a/ash/system/power/battery_notification.cc b/ash/system/power/battery_notification.cc
@@ -5,7 +5,6 @@
 #include "ash/system/power/battery_notification.h"
 
 #include "ash/public/cpp/power_utils.h"
-#include "ash/resources/grit/ash_resources.h"
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/strings/grit/ash_strings.h"
 #include "ash/system/power/power_status.h"

diff --git a/ash/system/power/dual_role_notification.cc b/ash/system/power/dual_role_notification.cc
@@ -6,7 +6,6 @@
 
 #include <set>
 
-#include "ash/resources/grit/ash_resources.h"
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/shell.h"
 #include "ash/strings/grit/ash_strings.h"

diff --git a/ash/system/power/peripheral_battery_notifier.cc b/ash/system/power/peripheral_battery_notifier.cc
@@ -6,7 +6,6 @@
 
 #include <vector>
 
-#include "ash/resources/grit/ash_resources.h"
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/shell.h"
 #include "ash/strings/grit/ash_strings.h"

diff --git a/ash/system/power/tray_power.cc b/ash/system/power/tray_power.cc
@@ -8,7 +8,7 @@
 
 #include "ash/accessibility/accessibility_delegate.h"
 #include "ash/public/cpp/ash_switches.h"
-#include "ash/resources/grit/ash_resources.h"
+
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/strings/grit/ash_strings.h"
 #include "ash/system/date/date_view.h"

diff --git a/ash/system/screen_layout_observer.cc b/ash/system/screen_layout_observer.cc
@@ -11,7 +11,6 @@
 #include "ash/display/screen_orientation_controller.h"
 #include "ash/metrics/user_metrics_action.h"
 #include "ash/metrics/user_metrics_recorder.h"
-#include "ash/resources/grit/ash_resources.h"
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/session/session_controller.h"
 #include "ash/shell.h"

diff --git a/ash/system/screen_security/screen_capture_tray_item.cc b/ash/system/screen_security/screen_capture_tray_item.cc
@@ -9,7 +9,7 @@
 #include "ash/metrics/user_metrics_action.h"
 #include "ash/metrics/user_metrics_recorder.h"
 #include "ash/public/cpp/ash_features.h"
-#include "ash/resources/grit/ash_resources.h"
+
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/shell.h"
 #include "ash/strings/grit/ash_strings.h"

diff --git a/ash/system/screen_security/screen_share_tray_item.cc b/ash/system/screen_security/screen_share_tray_item.cc
@@ -6,7 +6,6 @@
 
 #include <utility>
 
-#include "ash/resources/grit/ash_resources.h"
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/shell.h"
 #include "ash/strings/grit/ash_strings.h"

diff --git a/chrome/browser/chromeos/BUILD.gn b/chrome/browser/chromeos/BUILD.gn
@@ -21,7 +21,6 @@ source_set("chromeos") {
   configs += [ "//build/config/compiler:wexit_time_destructors" ]
 
   public_deps = [
-    "//ash/resources",
     "//ash/strings",
     "//chrome:extra_resources",
     "//chrome:resources",
@@ -2180,7 +2179,6 @@ source_set("unit_tests") {
     ":attestation_proto",
     ":test_support",
     "//ash",
-    "//ash/resources",
     "//base",
     "//chrome/common",
     "//chromeos/components/tether:test_support",

diff --git a/chrome/browser/ui/app_list/DEPS b/chrome/browser/ui/app_list/DEPS
@@ -6,10 +6,3 @@ include_rules = [
 
   "+ash/public",
 ]
-
-specific_include_rules = {
-  # TODO(733662): Remove when app_list is migrated.
-  "crostini_app_model_builder\.cc": [
-    "+ash/resources/grit/ash_resources.h",
-  ],
-}
diff --git a/chrome/browser/ui/app_list/crostini/crostini_app_model_builder.cc b/chrome/browser/ui/app_list/crostini/crostini_app_model_builder.cc
@@ -4,19 +4,16 @@
 
 #include "chrome/browser/ui/app_list/crostini/crostini_app_model_builder.h"
 
-#include "ash/resources/grit/ash_resources.h"
 #include "chrome/browser/chromeos/crostini/crostini_manager.h"
 #include "chrome/browser/chromeos/crostini/crostini_pref_names.h"
 #include "chrome/browser/chromeos/crostini/crostini_registry_service_factory.h"
 #include "chrome/browser/chromeos/crostini/crostini_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/app_list/app_list_controller_delegate.h"
 #include "chrome/browser/ui/app_list/crostini/crostini_app_item.h"
-#include "chrome/grit/chrome_unscaled_resources.h"
 #include "components/crx_file/id_util.h"
 #include "components/prefs/pref_change_registrar.h"
 #include "ui/base/l10n/l10n_util.h"
-#include "ui/base/resource/resource_bundle.h"
 
 CrostiniAppModelBuilder::CrostiniAppModelBuilder(
     AppListControllerDelegate* controller)

diff --git a/chrome/chrome_paks.gni b/chrome/chrome_paks.gni
@@ -59,12 +59,10 @@ template("chrome_repack_percent") {
     if (is_chromeos) {
       sources += [
         "$root_gen_dir/ash/components/resources/ash_components_resources_${percent}_percent.pak",
-        "$root_gen_dir/ash/resources/ash_resources_${percent}_percent.pak",
         "$root_gen_dir/ui/chromeos/resources/ui_chromeos_resources_${percent}_percent.pak",
       ]
       deps += [
         "//ash/components/resources",
-        "//ash/resources",
         "//ui/chromeos/resources",
       ]
     }

diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
@@ -3251,7 +3251,6 @@ test("unit_tests") {
     deps += [
       "//ash:test_support_with_content",
       "//ash/public/cpp/resources:ash_public_unscaled_resources",
-      "//ash/resources",
       "//ash/strings",
     ]
   }
@@ -4884,7 +4883,6 @@ if (!is_android) {
         "//ash:interactive_ui_test_support",
         "//ash/app_list/presenter:test_support",
         "//ash/public/interfaces:test_interfaces",
-        "//ash/resources",
         "//chrome/browser/media/router:test_support",
         "//chromeos",
         "//mojo/edk",

diff --git a/tools/check_grd_for_unused_strings.py b/tools/check_grd_for_unused_strings.py
@@ -137,7 +137,6 @@ def main():
     ui_chromeos_dir = os.path.join(ui_dir, 'chromeos')
     grd_files = [
       os.path.join(ash_base_dir, 'ash_strings.grd'),
-      os.path.join(ash_base_dir, 'resources', 'ash_resources.grd'),
       os.path.join(ash_components_dir, 'ash_components_strings.grd'),
       os.path.join(ash_components_dir, 'resources',
                    'ash_components_resources.grd'),

diff --git a/tools/gritsettings/resource_ids b/tools/gritsettings/resource_ids
@@ -293,9 +293,6 @@
   "ash/public/cpp/resources/ash_public_unscaled_resources.grd": {
     "includes": [24260],
   },
-  "ash/resources/ash_resources.grd": {
-    "structures": [24280],
-  },
   "ash/shell/ash_shell_resources.grd": {
     "includes": [24290],
   },

diff --git a/tools/resources/find_unused_resources.py b/tools/resources/find_unused_resources.py
@@ -12,7 +12,7 @@
 
 Example:
   cd /work/chrome/src
-  tools/resources/find_unused_resouces.py ash/resources/ash_resources.grd
+  tools/resources/find_unused_resouces.py chrome/browser/browser_resources.grd
 """
 
 __author__ = 'jamescook@chromium.org (James Cook)'