Title: [239224] trunk/Source/WebKit
Revision
239224
Author
joep...@webkit.org
Date
2018-12-14 11:36:54 -0800 (Fri, 14 Dec 2018)

Log Message

Unreviewed, apply post-landing review comments after r239221.

* UIProcess/WebPageDebuggable.cpp:
(WebKit::WebPageDebuggable::url const):
Switch to WTF::blankURL() instead of using "about:blank" directly.

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (239223 => 239224)


--- trunk/Source/WebKit/ChangeLog	2018-12-14 19:29:41 UTC (rev 239223)
+++ trunk/Source/WebKit/ChangeLog	2018-12-14 19:36:54 UTC (rev 239224)
@@ -1,3 +1,11 @@
+2018-12-14  Joseph Pecoraro  <pecor...@apple.com>
+
+        Unreviewed, apply post-landing review comments after r239221.
+
+        * UIProcess/WebPageDebuggable.cpp:
+        (WebKit::WebPageDebuggable::url const):
+        Switch to WTF::blankURL() instead of using "about:blank" directly.
+
 2018-12-14  Chris Dumez  <cdu...@apple.com>
 
         [PSON] Process-swapping on a loadHTMLString causes duplicate decidePolicyForNavigationAction delegate calls

Modified: trunk/Source/WebKit/UIProcess/WebPageDebuggable.cpp (239223 => 239224)


--- trunk/Source/WebKit/UIProcess/WebPageDebuggable.cpp	2018-12-14 19:29:41 UTC (rev 239223)
+++ trunk/Source/WebKit/UIProcess/WebPageDebuggable.cpp	2018-12-14 19:36:54 UTC (rev 239224)
@@ -55,11 +55,11 @@
 String WebPageDebuggable::url() const
 {
     if (!m_page.mainFrame())
-        return "about:blank"_s;
+        return WTF::blankURL().string();
 
     String url = ""
     if (url.isEmpty())
-        return "about:blank"_s;
+        return WTF::blankURL().string();
 
     return url;
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to