[devtools] Remove trailing fragment identifier from script urls

Source/core/fetch/ScriptResource.cpp

Index: Source/core/fetch/ScriptResource.cpp
diff --git a/Source/core/fetch/ScriptResource.cpp b/Source/core/fetch/ScriptResource.cpp
index 52bfa54b375880b382d9cfad66910ab34b146d9b..1632b02b3873a05017fac5fb6a0585f8a032d0df 100644
--- a/Source/core/fetch/ScriptResource.cpp
+++ b/Source/core/fetch/ScriptResource.cpp
@@ -84,6 +84,15 @@ const String& ScriptResource::script()
     return m_script.string();
 }
 
+const KURL& ScriptResource::scriptURL()
+{
+    if (!response().url().isEmpty() && m_url.isEmpty()) {
+        m_url = response().url();
+        m_url.removeFragmentIdentifier();
+    }
+    return m_url;

[abarth-chromium, 2013/10/17 01:06:55]

Why do we need to store this data?  Is it really too expensive to recompute it
each time?

[johnjbarton, 2013/10/17 02:30:11]

I believe we need to store this data so the const KURL& has something to point
to.
No, the only time we need to store is if we call scriptURL (already true) and if
we have a fragment id. So I could check the response().url() for a fragment,
update in that case, else return the response().url(). That way the
ScriptResource.m_url is an empty KURL except in the case we are using # URLs.

[abarth-chromium, 2013/10/17 02:39:05]

We can return a KURL rather than a const KURL&

+}
+
 bool ScriptResource::mimeTypeAllowedByNosniff() const
 {
     return parseContentTypeOptionsHeader(m_response.httpHeaderField("X-Content-Type-Options")) != ContentTypeOptionsNosniff || MIMETypeRegistry::isSupportedJavaScriptMIMEType(mimeType());