Settings People Revamp: Split Profile Info out into its own handler.

chrome/browser/resources/settings/people_page/profile_info_browser_proxy.js

+// Copyright 2016 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.
+
+/**
+ * @fileoverview A helper object used from the the People section to get the
+ * profile info, which consists of the profile name and icon. Used for both
+ * Chrome browser and ChromeOS.
+ */
+cr.exportPath('settings');
+
+/**
+ * An object describing the profile.
+ * @typedef {{
+ *   name: string,
+ *   iconUrl: string
+ * }}
+ */
+settings.ProfileInfo;
+
+cr.define('settings', function() {
+  /** @interface */
+  function ProfileInfoBrowserProxy() {}
+
+  ProfileInfoBrowserProxy.prototype = {
+    /**
+     * Returns a Promise for the profile info.
+     * @return {!Promise<!settings.ProfileInfo>}
+     */
+    getProfileInfo: function() {},
+  };
+
+  /**
+   * @constructor
+   * @implements {ProfileInfoBrowserProxy}
+   */
+  function ProfileInfoBrowserProxyImpl() {}
+  // The singleton instance_ is replaced with a test version of this wrapper

[dpapad, 2016/04/18 23:30:49]

Nit(optional): I think it is OK to not copy this comment into every
browser_proxy. It is already copied in quite a few such files.

[tommycli, 2016/04/19 00:03:26]

Done.

+  // during testing.
+  cr.addSingletonGetter(ProfileInfoBrowserProxyImpl);
+
+  ProfileInfoBrowserProxyImpl.prototype = {
+    /** @override */
+    getProfileInfo: function() {
+      return cr.sendWithPromise('getProfileInfo');
+    },
+  };
+
+  return {
+    ProfileInfoBrowserProxyImpl: ProfileInfoBrowserProxyImpl,
+  };
+});