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

Side by Side Diff: chrome/browser/resources/extensions/extension_list.js

Issue 10103015: "View website" instead of "View in Web Store" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update comment/var naming Created 8 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 | Annotate | Revision Log
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | chrome/browser/resources/extensions/extensions.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 cr.define('options', function() { 5 cr.define('options', function() {
6 'use strict'; 6 'use strict';
7 7
8 /** 8 /**
9 * A lookup helper function to find the first node that has an id (starting 9 * A lookup helper function to find the first node that has an id (starting
10 * at |node| and going up the parent chain). 10 * at |node| and going up the parent chain).
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
130 options.hidden = false; 130 options.hidden = false;
131 } 131 }
132 132
133 if (extension.allow_activity) { 133 if (extension.allow_activity) {
134 var activity = node.querySelector('.activity-link'); 134 var activity = node.querySelector('.activity-link');
135 activity.href = 'chrome://extension-activity?extensionId=' + 135 activity.href = 'chrome://extension-activity?extensionId=' +
136 extension.id; 136 extension.id;
137 activity.hidden = false; 137 activity.hidden = false;
138 } 138 }
139 139
140 // The 'View in Web Store' checkbox. 140 // The 'View in Web Store/View Web Site' link.
141 if (extension.homepageUrl) { 141 if (extension.homepageUrl) {
142 var store = node.querySelector('.store-link'); 142 var siteLink = node.querySelector('.site-link');
143 store.href = extension.homepageUrl; 143 siteLink.href = extension.homepageUrl;
144 store.hidden = false; 144 siteLink.textContent = localStrings.getString(
145 extension.homepageProvided ? 'extensionSettingsVisitWebsite' :
146 'extensionSettingsVisitWebStore');
147 siteLink.hidden = false;
145 } 148 }
146 149
147 // The 'Reload' checkbox. 150 // The 'Reload' checkbox.
148 if (extension.allow_reload) { 151 if (extension.allow_reload) {
149 var reload = node.querySelector('.reload-link'); 152 var reload = node.querySelector('.reload-link');
150 reload.addEventListener('click', function(e) { 153 reload.addEventListener('click', function(e) {
151 chrome.send('extensionSettingsReload', [extension.id]); 154 chrome.send('extensionSettingsReload', [extension.id]);
152 }); 155 });
153 reload.hidden = false; 156 reload.hidden = false;
154 } 157 }
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
243 } 246 }
244 247
245 this.appendChild(node); 248 this.appendChild(node);
246 }, 249 },
247 }; 250 };
248 251
249 return { 252 return {
250 ExtensionsList: ExtensionsList 253 ExtensionsList: ExtensionsList
251 }; 254 };
252 }); 255 });
OLDNEW
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | chrome/browser/resources/extensions/extensions.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698