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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/resources/settings/privacy_page/privacy_page.html

Issue 2554403005: [MD settings] move extra options in category settings (Closed)
Patch Set: renamed modules Created 4 years 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
« no previous file with comments | « no previous file | chrome/browser/resources/settings/settings_resources.grd » ('j') | chrome/browser/resources/settings/site_settings/category_default_setting.html » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/privacy_page/privacy_page.html
diff --git a/chrome/browser/resources/settings/privacy_page/privacy_page.html b/chrome/browser/resources/settings/privacy_page/privacy_page.html
index dbab2364e566f0b94af44e7f1cb113174dc56c38..1c519530b328f6aba2ee43d00bbdc7f4f862e9b1 100644
--- a/chrome/browser/resources/settings/privacy_page/privacy_page.html
+++ b/chrome/browser/resources/settings/privacy_page/privacy_page.html
@@ -14,12 +14,13 @@
<link rel="import" href="/settings_page/settings_subpage.html">
<link rel="import" href="/settings_shared_css.html">
<link rel="import" href="/site_settings/all_sites.html">
+<link rel="import" href="/site_settings/category_default_setting.html">
+<link rel="import" href="/site_settings/category_setting_exceptions.html">
<link rel="import" href="/site_settings/constants.html">
<link rel="import" href="/site_settings/media_picker.html">
<link rel="import" href="/site_settings/pdf_documents.html">
<link rel="import" href="/site_settings/protocol_handlers.html">
<link rel="import" href="/site_settings/site_data_details_subpage.html">
-<link rel="import" href="/site_settings/site_settings_category.html">
<link rel="import" href="/site_settings/usb_devices.html">
<link rel="import" href="/site_settings/site_data.html">
<link rel="import" href="/site_settings/zoom_levels.html">
@@ -216,10 +217,12 @@
<settings-subpage page-title="$i18n{siteSettingsAutomaticDownloads}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting
category="{{ContentSettingsTypes.AUTOMATIC_DOWNLOADS}}">
- </site-settings-category>
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
tommycli 2016/12/15 21:36:38 This can be in a followup CL, but reading the code
dschuyler 2016/12/16 20:39:45 Done.
+ category="{{ContentSettingsTypes.AUTOMATIC_DOWNLOADS}}">
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/backgroundSync" no-search>
@@ -227,10 +230,12 @@
<settings-subpage page-title="$i18n{siteSettingsBackgroundSync}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting
+ category="{{ContentSettingsTypes.BACKGROUND_SYNC}}">
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
category="{{ContentSettingsTypes.BACKGROUND_SYNC}}">
- </site-settings-category>
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/camera" no-search>
@@ -238,27 +243,31 @@
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
<media-picker type="camera"></media-picker>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting category="{{ContentSettingsTypes.CAMERA}}">
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
category="{{ContentSettingsTypes.CAMERA}}">
- </site-settings-category>
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/cookies" no-search>
<settings-subpage page-title="$i18n{siteSettingsCategoryCookies}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting category="{{ContentSettingsTypes.COOKIES}}"
+ sub-option-label="$i18n{deleteDataPostSession}"
+ sub-option-secondary=""><!-- To work with :empty. -->
+ </category-default-setting>
+ <div class="settings-box">
+ <settings-toggle-button class="start"
+ label="$i18n{thirdPartyCookie}"
+ sub-label="$i18n{thirdPartyCookieSublabel}"
+ pref="{{prefs.profile.block_third_party_cookies}}">
+ </settings-toggle-button>
+ </div>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
category="{{ContentSettingsTypes.COOKIES}}">
- <div class="settings-box extra-options">
- <settings-toggle-button class="start"
- label="$i18n{thirdPartyCookie}"
- sub-label="$i18n{thirdPartyCookieSublabel}"
- pref="{{prefs.profile.block_third_party_cookies}}">
- </settings-toggle-button>
- </div>
- </site-settings-category>
+ </category-setting-exceptions>
<site-data></site-data>
</settings-subpage>
</template>
@@ -266,20 +275,23 @@
<settings-subpage page-title="$i18n{siteSettingsCategoryImages}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting category="{{ContentSettingsTypes.IMAGES}}">
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
category="{{ContentSettingsTypes.IMAGES}}">
- </site-settings-category>
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/location" no-search>
<settings-subpage page-title="$i18n{siteSettingsCategoryLocation}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting
category="{{ContentSettingsTypes.GEOLOCATION}}">
- </site-settings-category>
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
+ category="{{ContentSettingsTypes.GEOLOCATION}}">
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/handlers" no-search>
@@ -301,20 +313,23 @@
<settings-subpage page-title="$i18n{siteSettingsCategoryJavascript}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting
+ category="{{ContentSettingsTypes.JAVASCRIPT}}">
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
category="{{ContentSettingsTypes.JAVASCRIPT}}">
- </site-settings-category>
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/keygen" no-search>
<settings-subpage page-title="$i18n{siteSettingsKeygen}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting category="{{ContentSettingsTypes.KEYGEN}}">
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
category="{{ContentSettingsTypes.KEYGEN}}">
- </site-settings-category>
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/microphone" no-search>
@@ -322,56 +337,66 @@
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
<media-picker type="mic"></media-picker>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting category="{{ContentSettingsTypes.MIC}}">
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
category="{{ContentSettingsTypes.MIC}}">
- </site-settings-category>
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/notifications" no-search>
<settings-subpage page-title="$i18n{siteSettingsCategoryNotifications}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting
+ category="{{ContentSettingsTypes.NOTIFICATIONS}}">
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
category="{{ContentSettingsTypes.NOTIFICATIONS}}">
- </site-settings-category>
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/flash" no-search>
<settings-subpage page-title="$i18n{siteSettingsFlash}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
- category="{{ContentSettingsTypes.PLUGINS}}">
+ <category-default-setting category="{{ContentSettingsTypes.PLUGINS}}"
+ sub-option-label="$i18n{siteSettingsFlashAskBefore}"
+ sub-option-secondary=
+ "$i18n{siteSettingsFlashAskBeforeSubtitle}">
+ </category-default-setting>
<if expr="chromeos">
- <div actionable class="settings-box extra-options"
- on-tap="onAdobeFlashStorageClicked_">
- <div class="list-item list-button">$i18n{adobeFlashStorage}</div>
- </div>
+ <div actionable class="settings-box"
+ on-tap="onAdobeFlashStorageClicked_">
+ <div class="list-item list-button">$i18n{adobeFlashStorage}</div>
+ </div>
</if>
- </site-settings-category>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
+ category="{{ContentSettingsTypes.PLUGINS}}">
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/popups" no-search>
<settings-subpage page-title="$i18n{siteSettingsCategoryPopups}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting category="{{ContentSettingsTypes.POPUPS}}">
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
category="{{ContentSettingsTypes.POPUPS}}">
- </site-settings-category>
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/unsandboxedPlugins" no-search>
<settings-subpage page-title="$i18n{siteSettingsUnsandboxedPlugins}">
<button class="icon-help subpage-title-extra"
is="paper-icon-button-light" on-tap="onHelpTap_"></button>
- <site-settings-category
- selected-site="{{selectedSite}}"
+ <category-default-setting
category="{{ContentSettingsTypes.UNSANDBOXED_PLUGINS}}">
- </site-settings-category>
+ </category-default-setting>
+ <category-setting-exceptions selected-site="{{selectedSite}}"
+ category="{{ContentSettingsTypes.UNSANDBOXED_PLUGINS}}">
+ </category-setting-exceptions>
</settings-subpage>
</template>
<template is="dom-if" route-path="/content/usbDevices" no-search>
@@ -384,8 +409,7 @@
<site-details site="[[selectedSite]]"></site-details>
</settings-subpage>
</template>
- <template is="dom-if" route-path="/cookies/detail"
- no-search>
+ <template is="dom-if" route-path="/cookies/detail" no-search>
<settings-subpage page-title="[[pageTitle]]">
<paper-button class="subpage-title-extra"
on-tap="onRemoveAllCookiesFromSite_">
« no previous file with comments | « no previous file | chrome/browser/resources/settings/settings_resources.grd » ('j') | chrome/browser/resources/settings/site_settings/category_default_setting.html » ('J')

Powered by Google App Engine
This is Rietveld 408576698