Change how DOM Inspector fetches document's base URL.

third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp

Index: third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp
diff --git a/third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp b/third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp
index 9fc19df1bd84172324427c68058d3c4ac872ea02..05030412f9312ef07f499dded9d952ad04a0c99b 100644
--- a/third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp
+++ b/third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp
@@ -1475,7 +1475,7 @@ String InspectorDOMAgent::documentURLString(Document* document)
 
 static String documentBaseURLString(Document* document)
 {
-    return document->completeURL("").string();
+    return document->baseURLForOverride(document->baseURL()).string();

[dgozman, 2015/12/16 18:00:49]

This change looks fine, but I'm not sure about baseURLForOverride naming and
changes in Document in general. Over to tkent@.

[mharanczyk, 2015/12/16 18:29:42]

I'm open for suggestions about naming, I'm not happy with it either.
I tried to preserve old code paths without duplicating code/logic, but maybe it
is enough to just return document->baseURL().string()? Doing empty string
completeURL magic can lead to some unfortunate failures when all Inspector wants
in baseURL.

[sof, 2015/12/16 18:53:27]

Will this not return the parent document's base url if document->baseURL() is
about:blank? that is not what's wanted here.

 }
 
 static TypeBuilder::DOM::ShadowRootType::Enum shadowRootType(ShadowRoot* shadowRoot)