Fix hard-coding of "about:srcdoc"

third_party/WebKit/Source/core/loader/FrameLoader.cpp

Index: third_party/WebKit/Source/core/loader/FrameLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/FrameLoader.cpp b/third_party/WebKit/Source/core/loader/FrameLoader.cpp
index 28d99d964d16b432e27c8878e60e5c31b3d20acc..3a5fa690bfb12454f58943e0c59ba1a7067b1b69 100644
--- a/third_party/WebKit/Source/core/loader/FrameLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/FrameLoader.cpp
@@ -1458,7 +1458,7 @@ bool FrameLoader::shouldTreatURLAsSameAsCurrent(const KURL& url) const
 
 bool FrameLoader::shouldTreatURLAsSrcdocDocument(const KURL& url) const
 {
-    if (!equalIgnoringCase(url.string(), "about:srcdoc"))
+    if (!equalIgnoringCase(url, srcdocURL()))

[Tom Sepez, 2015/11/16 17:16:06]

Shouldn't this be url.isAboutSrcDocUrl() ? We now have pair of KURL objects
here, it seems like we should be using a KURL method to make this comparison,
rather than letting them both implicitly convert to String and then using a
String method. Does KURL's normalization take care of the case issue for you?

[Xiaocheng, 2015/11/17 05:25:47]

Done.

         return false;
     HTMLFrameOwnerElement* ownerElement = m_frame->deprecatedLocalOwner();
     if (!isHTMLIFrameElement(ownerElement))