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

Side by Side Diff: chrome/browser/resources/settings/on_startup_page/startup_urls_page.js

Issue 1780053002: [MD settings] favicons for OnStartup url settings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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
« no previous file with comments | « chrome/browser/resources/settings/on_startup_page/startup_urls_page.html ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 * @typedef {{ 6 * @typedef {{
7 * 'title': string, 7 * 'title': string,
8 * 'tooltip': string, 8 * 'tooltip': string,
9 * 'url': string 9 * 'url': string
10 * }} 10 * }}
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 cr.define('Settings', function() { 53 cr.define('Settings', function() {
54 return { 54 return {
55 updateStartupPages: function() { 55 updateStartupPages: function() {
56 return self.updateStartupPages_.apply(self, arguments); 56 return self.updateStartupPages_.apply(self, arguments);
57 }, 57 },
58 }; 58 };
59 }); 59 });
60 chrome.send('onStartupPrefsPageLoad'); 60 chrome.send('onStartupPrefsPageLoad');
61 }, 61 },
62 62
63 getIconSet_: function(url) {
64 return getFaviconImageSet(url);
65 },
66
63 /** @private */ 67 /** @private */
64 updateStartupPages_: function(startupPages) { 68 updateStartupPages_: function(startupPages) {
65 this.startupPages_ = startupPages; 69 this.startupPages_ = startupPages;
66 }, 70 },
67 71
68 /** @private */ 72 /** @private */
69 onAddPageTap_: function() { 73 onAddPageTap_: function() {
70 this.newUrl_ = ''; 74 this.newUrl_ = '';
71 this.$.addUrlDialog.open(); 75 this.$.addUrlDialog.open();
72 }, 76 },
(...skipping 24 matching lines...) Expand all
97 }, 101 },
98 102
99 /** 103 /**
100 * @param {!{model: !{index: number}}} e 104 * @param {!{model: !{index: number}}} e
101 * @private 105 * @private
102 */ 106 */
103 onRemoveUrlTap_: function(e) { 107 onRemoveUrlTap_: function(e) {
104 chrome.send('removeStartupPage', [e.model.index]); 108 chrome.send('removeStartupPage', [e.model.index]);
105 }, 109 },
106 }); 110 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/settings/on_startup_page/startup_urls_page.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698