Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(368)

Side by Side Diff: chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_browser_proxy.js

Issue 2984843003: MD Settings: Convert all usages of .bind(this) to use ES6 arrow function. (Closed)
Patch Set: Fix compiler errors, newly revealed. Created 3 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview A helper object used from the "Clear browsing data" dialog 6 * @fileoverview A helper object used from the "Clear browsing data" dialog
7 * to interact with the browser. 7 * to interact with the browser.
8 */ 8 */
9 9
10 /** 10 /**
(...skipping 11 matching lines...) Expand all
22 * hasNotifications: boolean 22 * hasNotifications: boolean
23 * }} 23 * }}
24 */ 24 */
25 var ImportantSite; 25 var ImportantSite;
26 26
27 cr.define('settings', function() { 27 cr.define('settings', function() {
28 /** @interface */ 28 /** @interface */
29 class ClearBrowsingDataBrowserProxy { 29 class ClearBrowsingDataBrowserProxy {
30 /** 30 /**
31 * @param {!Array<!ImportantSite>} importantSites 31 * @param {!Array<!ImportantSite>} importantSites
32 * @return {!Promise<void>} 32 * @return {!Promise<boolean>}
dpapad 2017/07/22 00:00:44 Necessary to fix compiler error in clear_browsing_
33 * A promise resolved when data clearing has completed. 33 * A promise resolved when data clearing has completed. The boolean
34 * indicates whether an additional dialog should be shown, informing the
35 * user about other forms of browsing history.
34 */ 36 */
35 clearBrowsingData(importantSites) {} 37 clearBrowsingData(importantSites) {}
36 38
37 /** 39 /**
38 * @return {!Promise<!Array<!ImportantSite>>} 40 * @return {!Promise<!Array<!ImportantSite>>}
39 * A promise resolved when imporant sites are retrieved. 41 * A promise resolved when imporant sites are retrieved.
40 */ 42 */
41 getImportantSites() {} 43 getImportantSites() {}
42 44
43 /** 45 /**
(...skipping 23 matching lines...) Expand all
67 } 69 }
68 } 70 }
69 71
70 cr.addSingletonGetter(ClearBrowsingDataBrowserProxyImpl); 72 cr.addSingletonGetter(ClearBrowsingDataBrowserProxyImpl);
71 73
72 return { 74 return {
73 ClearBrowsingDataBrowserProxy: ClearBrowsingDataBrowserProxy, 75 ClearBrowsingDataBrowserProxy: ClearBrowsingDataBrowserProxy,
74 ClearBrowsingDataBrowserProxyImpl: ClearBrowsingDataBrowserProxyImpl, 76 ClearBrowsingDataBrowserProxyImpl: ClearBrowsingDataBrowserProxyImpl,
75 }; 77 };
76 }); 78 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698