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

Unified Diff: chrome/browser/resources/options/browser_options.js

Issue 2121303003: Remove the "Settings" packaged app (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/options/browser_options.js
diff --git a/chrome/browser/resources/options/browser_options.js b/chrome/browser/resources/options/browser_options.js
index 8d9e03201e7d90e43e45cd639635b275d7dce052..1fdaa552dd474bb4e408a8b350880693415d4d34 100644
--- a/chrome/browser/resources/options/browser_options.js
+++ b/chrome/browser/resources/options/browser_options.js
@@ -389,12 +389,6 @@ cr.define('options', function() {
['Options_ShowCreateProfileDlg']);
ManageProfileOverlay.showCreateDialog();
};
- if (OptionsPage.isSettingsApp()) {
- $('profiles-app-list-switch').onclick = function(event) {
- var selectedProfile = self.getSelectedProfileItem_();
- chrome.send('switchAppListProfile', [selectedProfile.filePath]);
- };
- }
$('profiles-manage').onclick = function(event) {
chrome.send('metricsHandler:recordAction',
['Options_ShowEditProfileDlg']);
@@ -483,7 +477,6 @@ cr.define('options', function() {
PageManager.showPageByName('clearBrowserData');
chrome.send('coreOptionsUserMetricsAction', ['Options_ClearData']);
};
- $('privacyClearDataButton').hidden = OptionsPage.isSettingsApp();
if ($('metrics-reporting-enabled')) {
$('metrics-reporting-enabled').checked =
@@ -1527,10 +1520,6 @@ cr.define('options', function() {
$('profiles-manage').title = '';
$('profiles-delete').disabled = !profilesList.canDeleteItems ||
!hasSelection;
- if (OptionsPage.isSettingsApp()) {
- $('profiles-app-list-switch').disabled = !hasSelection ||
- selectedProfile.isCurrentProfile;
- }
var importData = $('import-data');
if (importData) {
importData.disabled = $('import-data').disabled = hasSelection &&
@@ -1550,13 +1539,10 @@ cr.define('options', function() {
var showSingleProfileView = !usingNewProfilesUI && numProfiles == 1;
$('profiles-list').hidden = showSingleProfileView;
$('profiles-single-message').hidden = !showSingleProfileView;
- $('profiles-manage').hidden =
- showSingleProfileView || OptionsPage.isSettingsApp();
+ $('profiles-manage').hidden = showSingleProfileView;
$('profiles-delete').textContent = showSingleProfileView ?
loadTimeData.getString('profilesDeleteSingle') :
loadTimeData.getString('profilesDelete');
- if (OptionsPage.isSettingsApp())
- $('profiles-app-list-switch').hidden = showSingleProfileView;
},
/**
« no previous file with comments | « chrome/browser/resources/options/browser_options.html ('k') | chrome/browser/resources/options/options.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698