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

Unified Diff: chrome/test/data/devtools/target_list/background.js

Issue 46523002: Fix Telemetry. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 2 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
« no previous file with comments | « chrome/chrome.gyp ('k') | content/browser/devtools/devtools_http_handler_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/devtools/target_list/background.js
diff --git a/chrome/test/data/devtools/target_list/background.js b/chrome/test/data/devtools/target_list/background.js
index 60a6ca072c988215a837529dee7f90968daecd00..607f1e55d7f1db3693eb46271583a3d5fff710e3 100644
--- a/chrome/test/data/devtools/target_list/background.js
+++ b/chrome/test/data/devtools/target_list/background.js
@@ -32,7 +32,7 @@ function checkTarget(targets, url, type, opt_title, opt_faviconUrl) {
target.devtoolsFrontendUrl);
// On some platforms (e.g. Chrome OS) target.faviconUrl might be empty for
// a freshly created tab. Ignore the check then.
- if (target.faviconUrl && opt_faviconUrl)
+ if (target.faviconUrl)
chrome.test.assertEq(opt_faviconUrl, target.faviconUrl);
// Sometimes thumbnailUrl is not available for a freshly loaded tab.
if (target.thumbnailUrl)
@@ -56,8 +56,7 @@ chrome.test.runTests([
chrome.extension.getURL('favicon.png'));
checkTarget(targets,
chrome.extension.getURL('_generated_background_page.html'),
- 'background_page',
- 'Remote Debugger Test');
+ 'other');
chrome.test.succeed();
});
« no previous file with comments | « chrome/chrome.gyp ('k') | content/browser/devtools/devtools_http_handler_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698