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

Unified Diff: Source/devtools/front_end/sdk/SASSSourceMapping.js

Issue 298333003: DevTools: Implement console message logging through an extension (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Remove duplicate logging 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: Source/devtools/front_end/sdk/SASSSourceMapping.js
diff --git a/Source/devtools/front_end/sdk/SASSSourceMapping.js b/Source/devtools/front_end/sdk/SASSSourceMapping.js
index 98c5028302dd01779b38d1e12c29d62b4b288759..7c48fb7bcde07c356ead9e56d19fb409dc9c24e8 100644
--- a/Source/devtools/front_end/sdk/SASSSourceMapping.js
+++ b/Source/devtools/front_end/sdk/SASSSourceMapping.js
@@ -141,7 +141,7 @@ WebInspector.SASSSourceMapping.prototype = {
var etagMessage = this._headerValue("etag", headers) ? ", \"ETag\" response header found instead" : "";
var message = String.sprintf("The \"Last-Modified\" response header is missing or invalid for %s%s. The CSS auto-reload functionality will not work correctly.", url, etagMessage);
- WebInspector.console.log(message);
+ this._cssModel.target().consoleModel.log(message);
return null;
},
@@ -242,7 +242,7 @@ WebInspector.SASSSourceMapping.prototype = {
{
var cssUISourceCode = this._workspace.uiSourceCodeForURL(cssURL);
if (!cssUISourceCode) {
- WebInspector.console.log(WebInspector.UIString("%s resource missing. Please reload the page.", cssURL));
+ WebInspector.messageSink.addMessage(WebInspector.UIString("%s resource missing. Please reload the page.", cssURL));
callback(cssURL, sassURL, true);
return;
}
« no previous file with comments | « Source/devtools/front_end/sdk/LiveEditSupport.js ('k') | Source/devtools/front_end/sdk/StylesSourceMapping.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698