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

Unified Diff: src/messages.js

Issue 1044173002: [V8] Don't ignore sourceURL comment in inline scripts in .stack (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 9 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 | « no previous file | test/cctest/test-api.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/messages.js
diff --git a/src/messages.js b/src/messages.js
index 672fd4b8100fa26b499d983ab0d06a379d435793..fffc91d096ec96d9ead9f54987e349449366ee90 100644
--- a/src/messages.js
+++ b/src/messages.js
@@ -589,8 +589,8 @@ function ScriptLineCount() {
/**
- * If sourceURL comment is available and script starts at zero returns sourceURL
- * comment contents. Otherwise, script name is returned. See
+ * If sourceURL comment is available returns sourceURL comment contents.
+ * Otherwise, script name is returned. See
* http://fbug.googlecode.com/svn/branches/firebug1.1/docs/ReleaseNotes_1.1.txt
* and Source Map Revision 3 proposal for details on using //# sourceURL and
* deprecated //@ sourceURL comment to identify scripts that don't have name.
@@ -599,12 +599,7 @@ function ScriptLineCount() {
* deprecated //@ sourceURL comment otherwise.
*/
function ScriptNameOrSourceURL() {
- if (this.line_offset > 0 || this.column_offset > 0) {
- return this.name;
- }
- if (this.source_url) {
- return this.source_url;
- }
+ if (this.source_url) return this.source_url;
return this.name;
}
« no previous file with comments | « no previous file | test/cctest/test-api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698