[devtools] Remove trailing fragment identifier from script urls

Source/bindings/v8/ScriptDebugServer.cpp

Index: Source/bindings/v8/ScriptDebugServer.cpp
diff --git a/Source/bindings/v8/ScriptDebugServer.cpp b/Source/bindings/v8/ScriptDebugServer.cpp
index 34f6bc7cbe1b7586095a48f27a737441d5d07771..a15da89bc3ed6adad034e0e5308624f32971b551 100644
--- a/Source/bindings/v8/ScriptDebugServer.cpp
+++ b/Source/bindings/v8/ScriptDebugServer.cpp
@@ -502,7 +502,16 @@ void ScriptDebugServer::dispatchDidParseSource(ScriptDebugListener* listener, v8
     String sourceID = toWebCoreStringWithUndefinedOrNullCheck(object->Get(v8::String::NewSymbol("id")));
 
     ScriptDebugListener::Script script;
-    script.url = toWebCoreStringWithUndefinedOrNullCheck(object->Get(v8::String::NewSymbol("name")));
+
+    String rawScriptURL =  toWebCoreStringWithUndefinedOrNullCheck(object->Get(v8::String::NewSymbol("name")));
+    // Remove any fragment identifier.
+    size_t hashCharPos = rawScriptURL.find("#");

[abarth-chromium, 2013/10/11 20:20:41]

This seems like the wrong layer for this work.  The job of the bindings layer is
just to do type conversions.  All "application logic" should be handled outside
the bindings.

Also, shouldn't we use a real URL parser?

[johnjbarton, 2013/10/11 21:31:16]

The debugger layer (listener here) gets a const Script&. I could change
didParseSource() to take Script&, but that seems wrong.

The other place I can fix it is before we compile. Two classes represent JS it
seems, one is ScriptSourceCode but that is still in bindings. The other is
WebScriptSource. Construction of WebScriptSource from a string and a URL could
remove the fragment identifier.
WebScriptSource has a WebURL with a path to GURL and GURL::PathForRequest()
which computes the string we would want to use for scripts.

+    if (hashCharPos != kNotFound) {
+        script.url = rawScriptURL.substring(0, hashCharPos);
+    } else {
+        script.url = rawScriptURL;
+    }

[abarth-chromium, 2013/10/11 20:20:41]

This branch isn't needed.  substring(0, kNotFound) will just return the whole
string without making a copy.

[johnjbarton, 2013/10/11 21:31:16]

If I understand you, I could have written:
   script.url = script.url.substring(0, script.url.find("#"));
?

+
     script.source = toWebCoreStringWithUndefinedOrNullCheck(object->Get(v8::String::NewSymbol("source")));
     script.sourceMappingURL = toWebCoreStringWithUndefinedOrNullCheck(object->Get(v8::String::NewSymbol("sourceMappingURL")));
     script.startLine = object->Get(v8::String::NewSymbol("startLine"))->ToInteger()->Value();