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

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

Issue 293063017: Updated to read remove all shown when the user searches for a cookie (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Inserted the lines in cookies_view.js Created 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/options/cookies_view.js
diff --git a/chrome/browser/resources/options/cookies_view.js b/chrome/browser/resources/options/cookies_view.js
index 4d496684c428f87e17841876cdf7252ac3be9fa6..ba78a0da0e23c85af84bbc1d21eb84a3a47c3d35 100644
--- a/chrome/browser/resources/options/cookies_view.js
+++ b/chrome/browser/resources/options/cookies_view.js
@@ -88,6 +88,15 @@ cr.define('options', function() {
* @private
*/
handleSearchQueryChange_: function(e) {
+ var content = document.querySelector('.cookies-search-box').value;
+ if (content) {
+ document.querySelector('.remove-all-cookies-button').innerHTML =
+ loadTimeData.getString('remove_all_shown_cookie');
+ }
+ else {
+ document.querySelector('.remove-all-cookies-button').innerHTML =
+ loadTimeData.getString('remove_all_cookie');
+ }
Dan Beam 2014/05/24 02:03:52 i think you can shave some lines with this: var s
Dan Beam 2014/05/27 21:04:31 what happened to re-writing the code like this? ^
if (this.queryDelayTimerId_)
window.clearTimeout(this.queryDelayTimerId_);

Powered by Google App Engine
This is Rietveld 408576698