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

Unified Diff: chrome/browser/resources/options/clear_browser_data_overlay.html

Issue 2594723002: Count number of origins with data affected by clearing "cookies and site data". (Closed)
Patch Set: Add comments for issues with incomplete data deletion Created 3 years, 11 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/clear_browser_data_overlay.html
diff --git a/chrome/browser/resources/options/clear_browser_data_overlay.html b/chrome/browser/resources/options/clear_browser_data_overlay.html
index a36ef9069db4675ecd35a41f412678e49c0c35dd..303a73c2619208d7af08a2a87841b0d9702046cf 100644
--- a/chrome/browser/resources/options/clear_browser_data_overlay.html
+++ b/chrome/browser/resources/options/clear_browser_data_overlay.html
@@ -40,11 +40,16 @@
<div id="delete-cookies-container" class="checkbox">
<label>
<input id="delete-cookies-checkbox"
- pref="browser.clear_data.cookies" type="checkbox">
+ pref="browser.clear_data.cookies" type="checkbox"
+ aria-controls="delete-cookies-counter">
msramek 2017/01/09 12:54:44 I don't think it was the right decision, but anywa
dullweber 2017/01/09 16:05:46 Ok, I reverted this
<span i18n-content="deleteCookiesFlashCheckbox"
class="clear-plugin-lso-data-enabled"></span>
- <span i18n-content="deleteCookiesCheckbox"
- class="clear-plugin-lso-data-disabled"></span>
+ <span>
+ <span i18n-content="deleteCookiesCheckbox"
+ class="clear-plugin-lso-data-disabled"></span>
+ <span class="clear-browser-data-counter" role="note"
+ aria-live="polite" id="delete-cookies-counter"></span>
+ </span>
</label>
</div>
<div id="delete-cache-container" class="checkbox">

Powered by Google App Engine
This is Rietveld 408576698