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

Side by Side Diff: chrome/browser/resources/settings/site_settings/site_settings_behavior.js

Issue 2847963003: [MD settings] specify @override on created, ready, and attached (Closed)
Patch Set: Created 3 years, 7 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 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 * @fileoverview Behavior common to Site Settings classes. 6 * @fileoverview Behavior common to Site Settings classes.
7 */ 7 */
8 8
9 9
10 /** 10 /**
(...skipping 21 matching lines...) Expand all
32 category: String, 32 category: String,
33 33
34 /** 34 /**
35 * The browser proxy used to retrieve and change information about site 35 * The browser proxy used to retrieve and change information about site
36 * settings categories and the sites within. 36 * settings categories and the sites within.
37 * @type {settings.SiteSettingsPrefsBrowserProxy} 37 * @type {settings.SiteSettingsPrefsBrowserProxy}
38 */ 38 */
39 browserProxy: Object, 39 browserProxy: Object,
40 }, 40 },
41 41
42 /** @override */
42 created: function() { 43 created: function() {
43 this.browserProxy = 44 this.browserProxy =
44 settings.SiteSettingsPrefsBrowserProxyImpl.getInstance(); 45 settings.SiteSettingsPrefsBrowserProxyImpl.getInstance();
45 }, 46 },
46 47
48 /** @override */
47 ready: function() { 49 ready: function() {
48 this.PermissionValues = settings.PermissionValues; 50 this.PermissionValues = settings.PermissionValues;
49 }, 51 },
50 52
51 /** 53 /**
52 * Ensures the URL has a scheme (assumes http if omitted). 54 * Ensures the URL has a scheme (assumes http if omitted).
53 * @param {string} url The URL with or without a scheme. 55 * @param {string} url The URL with or without a scheme.
54 * @return {string} The URL with a scheme, or an empty string. 56 * @return {string} The URL with a scheme, or an empty string.
55 */ 57 */
56 ensureUrlHasScheme: function(url) { 58 ensureUrlHasScheme: function(url) {
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
181 setting: exception.setting, 183 setting: exception.setting,
182 enforcement: enforcement, 184 enforcement: enforcement,
183 controlledBy: controlledBy, 185 controlledBy: controlledBy,
184 }; 186 };
185 }, 187 },
186 188
187 }; 189 };
188 190
189 /** @polymerBehavior */ 191 /** @polymerBehavior */
190 var SiteSettingsBehavior = [SiteSettingsBehaviorImpl]; 192 var SiteSettingsBehavior = [SiteSettingsBehaviorImpl];
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698