MD Settings: validate home button url input

chrome/browser/resources/settings/appearance_page/appearance_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.
 
 cr.define('settings', function() {
   /** @interface */
   function AppearanceBrowserProxy() {}
 
   AppearanceBrowserProxy.prototype = {
     /** @return {!Promise<number>} */
     getDefaultZoom: assertNotReached,
 
     /**
      * @param {string} themeId
      * @return {!Promise<!chrome.management.ExtensionInfo>} Theme info.
      */
     getThemeInfo: assertNotReached,
 
     /** @return {boolean} Whether the current profile is supervised. */
     isSupervised: assertNotReached,
 
-// <if expr="chromeos">
+    // <if expr="chromeos">
     openWallpaperManager: assertNotReached,
-// </if>
+    // </if>
 
     useDefaultTheme: assertNotReached,
 
-// <if expr="is_linux and not chromeos">
+    // <if expr="is_linux and not chromeos">
     useSystemTheme: assertNotReached,
-// </if>
+    // </if>
+
+    /**
+     * @param {string} url The url of which to check validity.
+     * @return {!Promise<boolean>}
+     */
+    validateStartupPage: assertNotReached

[dpapad, 2017/03/24 21:47:43]

Nit (optional): It's OK to add a trailing comma here.

[scottchen, 2017/03/24 22:56:31]

Done.

   };
 
   /**
    * @implements {settings.AppearanceBrowserProxy}
    * @constructor
    */
   function AppearanceBrowserProxyImpl() {}
 
   cr.addSingletonGetter(AppearanceBrowserProxyImpl);
 
@@ skipping 10 matching lines @@
       return new Promise(function(resolve) {
         chrome.management.get(themeId, resolve);
       });
     },
 
     /** @override */
     isSupervised: function() {
       return loadTimeData.getBoolean('isSupervised');
     },
 
-// <if expr="chromeos">
+    // <if expr="chromeos">
     /** @override */
     openWallpaperManager: function() {
       chrome.send('openWallpaperManager');
     },
-// </if>
+    // </if>
 
     /** @override */
     useDefaultTheme: function() {
       chrome.send('useDefaultTheme');
     },
 
-// <if expr="is_linux and not chromeos">
+    // <if expr="is_linux and not chromeos">
     /** @override */
     useSystemTheme: function() {
       chrome.send('useSystemTheme');
     },
-// </if>
+    // </if>
+
+    /** @override */
+    validateStartupPage: function(url) {
+      return cr.sendWithPromise('validateStartupPage', url);
+    },
   };
 
   return {
     AppearanceBrowserProxy: AppearanceBrowserProxy,
     AppearanceBrowserProxyImpl: AppearanceBrowserProxyImpl,
   };
 });