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

Unified Diff: chrome/browser/resources/sync_internals/sync_search.js

Issue 111833002: Add missing argument to Element.setAttribute() calls (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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
Index: chrome/browser/resources/sync_internals/sync_search.js
diff --git a/chrome/browser/resources/sync_internals/sync_search.js b/chrome/browser/resources/sync_internals/sync_search.js
index d912046c71377c855e3220cc3aacee8bc0f2b5bb..9ed2456700b8a54d182a11c85acabd9781cabe3a 100644
--- a/chrome/browser/resources/sync_internals/sync_search.js
+++ b/chrome/browser/resources/sync_internals/sync_search.js
@@ -63,7 +63,7 @@ cr.define('chrome.sync', function() {
doSearch(query, function(nodes, error) {
if (error) {
statusControl.textContent = 'Error: ' + error;
- queryControl.setAttribute('error');
+ queryControl.setAttribute('error', '');
} else {
statusControl.textContent =
'Found ' + nodes.length + ' nodes in ' +
« no previous file with comments | « chrome/browser/resources/sync_internals/about.js ('k') | native_client_sdk/src/examples/api/file_io/example.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698