Fetch ARC Kiosk app name and icon from Android side.

chrome/browser/chromeos/app_mode/kiosk_app_icon_loader.h

+// 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_CHROMEOS_APP_MODE_KIOSK_APP_ICON_LOADER_H_
+#define CHROME_BROWSER_CHROMEOS_APP_MODE_KIOSK_APP_ICON_LOADER_H_
+
+#include "base/files/file_path.h"
+#include "base/memory/ref_counted_memory.h"
+#include "base/memory/weak_ptr.h"
+#include "base/sequenced_task_runner.h"
+#include "ui/gfx/image/image_skia.h"
+
+namespace chromeos {
+
+// Loads locally stored icon data and decodes it.
+class KioskAppIconLoader {
+ public:
+  enum LoadResult {
+    SUCCESS,
+    FAILED_TO_LOAD,
+    FAILED_TO_DECODE,
+  };
+
+  class Delegate {
+   public:
+    Delegate() = default;
+
+    virtual void OnIconLoadSuccess(const gfx::ImageSkia& icon) = 0;
+    virtual void OnIconLoadFailure() = 0;
+
+   protected:
+    virtual ~Delegate() = default;
+
+   private:
+    DISALLOW_COPY_AND_ASSIGN(Delegate);
+  };
+
+  KioskAppIconLoader(const base::WeakPtr<Delegate>& client,
+                     const base::FilePath& icon_path);
+
+  void Start();
+
+ private:
+  friend class base::RefCountedThreadSafe<KioskAppIconLoader>;
+
+  class IconImageRequest;
+
+  ~KioskAppIconLoader();
+
+  // Loads the icon from locally stored |icon_path_| on the blocking pool
+  void LoadOnBlockingPool();
+  void ReportResultOnBlockingPool(LoadResult result);
+  void NotifyClient();
+  void ReportResultOnUIThread();
+
+  base::WeakPtr<Delegate> client_;

[xiyuan, 2017/03/28 16:50:09]

nit: client_ -> delegate_ or Delegate -> Client to make it consistent.

[Sergey Poromov, 2017/03/29 16:03:44]

Done.

+  base::FilePath icon_path_;

[xiyuan, 2017/03/28 16:50:09]

nit: Make LoadOnBlockingPool take a const base::FilePath& and get rid of this.

[Sergey Poromov, 2017/03/29 16:03:44]

Done.

+
+  LoadResult load_result_;

[xiyuan, 2017/03/28 16:50:10]

nit: Make NotifyClient take a LoadResult and get rid of this.

[Sergey Poromov, 2017/03/29 16:03:44]

Done.

+  scoped_refptr<base::SequencedTaskRunner> task_runner_;
+
+  gfx::ImageSkia icon_;
+  scoped_refptr<base::RefCountedString> raw_icon_;
+
+  DISALLOW_COPY_AND_ASSIGN(KioskAppIconLoader);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_APP_MODE_KIOSK_APP_ICON_LOADER_H_