Commit 68e5274a authored by finnur@chromium.org's avatar finnur@chromium.org

Revert 102101 - Displaying the extensions count in extensions settings page.

(Vetoed by the UI leads)
BUG=96855TEST=Load chrome://settings/extensionSettings and see that the title format is now Extensions (xx) where xx is the number of installed extensions in the list. Also if there are no extensions loaded then the title should be Extensions.Patch by naveenbobbili@motorola.comReview URL: http://codereview.chromium.org/7932021TBR=naveenbobbili@motorola.com
Review URL: http://codereview.chromium.org/7993001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102267 0039d316-1c4b-4281-b951-d872f2087c98
parent ec8984f5
...@@ -123,5 +123,4 @@ Mark Seaborn <mrs@mythic-beasts.com> ...@@ -123,5 +123,4 @@ Mark Seaborn <mrs@mythic-beasts.com>
Amruth Raj <amruthraj@motorola.com> Amruth Raj <amruthraj@motorola.com>
Gajendra Singh <wxjg68@motorola.com> Gajendra Singh <wxjg68@motorola.com>
Ehsan Akhgari <ehsan.akhgari@gmail.com> Ehsan Akhgari <ehsan.akhgari@gmail.com>
Naveen Bobbili <naveenbobbili@motorola.com>
Christopher Dale <chrelad@gmail.com> Christopher Dale <chrelad@gmail.com>
<div id="extension-settings" class="page" hidden> <div id="extension-settings" class="page" hidden>
<h1 id="extension-settings-header" i18n-content="extensionSettingsTitle" <h1 i18n-content="extensionSettingsTitle"></h1>
hidden="true"></h1> <div id="dev-toggle">
<div id="dev-toggle" hidden="true">
<input id="toggle-dev-on" type="checkbox" value="off"></input> <input id="toggle-dev-on" type="checkbox" value="off"></input>
<label for="toggle-dev-on" i18n-content="extensionSettingsDeveloperMode" /> <label for="toggle-dev-on" i18n-content="extensionSettingsDeveloperMode" />
</div> </div>
......
...@@ -153,19 +153,6 @@ cr.define('options', function() { ...@@ -153,19 +153,6 @@ cr.define('options', function() {
$('suggest-gallery').hidden = true; $('suggest-gallery').hidden = true;
$('get-more-extensions-container').hidden = true; $('get-more-extensions-container').hidden = true;
// Append extension count to extensionSettingsTitle and frame
// the extension header string. If there are no extensions loaded
// then display title as Extensions only.
$('extension-settings-header').hidden = false;
$('dev-toggle').hidden = false;
var extensionSettingsHeader = templateData.extensionSettingsTitle;
if (extensionsData.extensions.length > 0) {
extensionSettingsHeader = templateData.extensionSettingsTitle + " (" +
extensionsData.extensions.length + ")";
}
$('extension-settings-header').innerText = extensionSettingsHeader;
if (extensionsData.extensions.length > 0) { if (extensionsData.extensions.length > 0) {
// Enforce order specified in the data or (if equal) then sort by // Enforce order specified in the data or (if equal) then sort by
// extension name (case-insensitive). // extension name (case-insensitive).
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment