chrome/browser/chromeos/file_system_provider/notification_manager.cc - Issue 2819413003: Refactor extension app icon.

Unified Diff: chrome/browser/chromeos/file_system_provider/notification_manager.cc

Issue 2819413003: Refactor extension app icon. (Closed)

Patch Set: rebase + nit Created 3 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 side-by-side diff with in-line comments

« chrome/browser/chromeos/extensions/gfx_utils.cc ('K') | « chrome/browser/chromeos/extensions/gfx_utils.cc ('k') | chrome/browser/extensions/BUILD.gn » ('j') | chrome/browser/extensions/api/file_system/request_file_system_notification.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/browser/chromeos/file_system_provider/notification_manager.cc

diff --git a/chrome/browser/chromeos/file_system_provider/notification_manager.cc b/chrome/browser/chromeos/file_system_provider/notification_manager.cc

index ece689b192a275fb39af19c8c93f3fbf624f4664..bfb8648d94adc0490c82ac50c087c7ae5996b080 100644

--- a/chrome/browser/chromeos/file_system_provider/notification_manager.cc

+++ b/chrome/browser/chromeos/file_system_provider/notification_manager.cc

@@ -7,7 +7,7 @@

#include "base/macros.h"

#include "base/strings/utf_string_conversions.h"

#include "chrome/browser/browser_process.h"

-#include "chrome/browser/extensions/extension_app_icon_loader.h"

+#include "chrome/browser/extensions/chrome_app_icon_loader.h"

#include "chrome/browser/ui/ash/multi_user/multi_user_util.h"

#include "chrome/grit/generated_resources.h"

#include "components/signin/core/account_id/account_id.h"

@@ -56,7 +56,7 @@ NotificationManager::NotificationManager(

: profile_(profile),

file_system_info_(file_system_info),

icon_loader_(

msw 2017/05/03 01:17:37 optional nit/aside: icon_loader_ need not be a std

khmel 2017/05/03 02:15:21 If you don't mind I won't change. This change is j

msw 2017/05/08 19:59:53 That's fine, it's tangential to your change.

khmel 2017/05/09 00:05:03 Acknowledged.

- new extensions::ExtensionAppIconLoader(profile, kIconSize, this)) {}

+ new extensions::ChromeAppIconLoader(profile, kIconSize, this)) {}

NotificationManager::~NotificationManager() {

if (callbacks_.size()) {