Refactor extension app icon.

chrome/browser/extensions/extension_app_icon.h

Index: chrome/browser/extensions/extension_app_icon.h
diff --git a/chrome/browser/extensions/extension_app_icon.h b/chrome/browser/extensions/extension_app_icon.h
new file mode 100644
index 0000000000000000000000000000000000000000..da9e5011731af531a1a220d01941b40c0b06f52e
--- /dev/null
+++ b/chrome/browser/extensions/extension_app_icon.h
@@ -0,0 +1,71 @@
+// Copyright 2017 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.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_APP_ICON_H_
+#define CHROME_BROWSER_EXTENSIONS_EXTENSION_APP_ICON_H_
+
+#include <memory>
+#include <string>
+
+#include "base/macros.h"
+#include "base/memory/weak_ptr.h"
+#include "extensions/browser/extension_icon_image.h"
+
+namespace content {
+class BrowserContext;
+}
+
+namespace extensions {
+
+class Extension;
+class ExtensionAppIconDelegate;
+class ExtensionAppIconService;
+class IconImage;
+
+class ExtensionAppIcon : public extensions::IconImage::Observer {

[xiyuan, 2017/04/18 20:43:00]

Document the class

[khmel, 2017/04/20 00:13:37]

Done.

+ public:
+  ExtensionAppIcon(const base::WeakPtr<ExtensionAppIconService>& host,
+                   ExtensionAppIconDelegate* delegate,
+                   content::BrowserContext* context,
+                   const std::string& app_id,
+                   int resource_size_in_dip);
+  ~ExtensionAppIcon() override;
+
+  void Reload();
+
+  // Returns true if icon is loaded and attached to valid extension.
+  bool IsValid() const;
+
+  void UpdateIcon();
+
+  void EnsureRepsForSupportedScales();
+
+  const std::string& app_id() const { return app_id_; }
+  content::BrowserContext* context() { return context_; }
+
+ private:
+  const extensions::Extension* GetExtension();
+
+  // extensions::IconImage::Observer:
+  void OnExtensionIconImageChanged(extensions::IconImage* image) override;
+  void OnExtensionIconImageDestroyed(extensions::IconImage* image) override;
+
+  base::WeakPtr<ExtensionAppIconService> host_;
+
+  // Unowned pointers.
+  ExtensionAppIconDelegate* const delegate_;
+  content::BrowserContext* const context_;
+
+  const std::string app_id_;
+
+  const int resource_size_in_dip_;
+
+  std::unique_ptr<extensions::IconImage> icon_;
+
+  DISALLOW_COPY_AND_ASSIGN(ExtensionAppIcon);
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_APP_ICON_H_