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

Unified Diff: src/inspector/v8-debugger-script.cc

Issue 2730813002: [inspector] fixed endLine in Debugger.scriptParsed event (Closed)
Patch Set: Created 3 years, 10 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/inspector/debugger/script-end-location.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/inspector/v8-debugger-script.cc
diff --git a/src/inspector/v8-debugger-script.cc b/src/inspector/v8-debugger-script.cc
index f7ddd4d697759d1cd2109445f5da588a67912c91..d4f94dd15a238ce5ea4140f4c0e5490315d4804c 100644
--- a/src/inspector/v8-debugger-script.cc
+++ b/src/inspector/v8-debugger-script.cc
@@ -134,11 +134,6 @@ class ActualScript : public V8DebuggerScript {
if (script->Source().ToLocal(&tmp)) {
m_source = toProtocolString(tmp);
- // V8 will not count last line if script source ends with \n.
- if (m_source.length() > 0 && m_source[m_source.length() - 1] == '\n') {
- m_endLine++;
- m_endColumn = 0;
- }
}
m_isModule = script->IsModule();
« no previous file with comments | « no previous file | test/inspector/debugger/script-end-location.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698