MiniBrowser should be able to navigate to about:blank
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Jan 2019 04:39:23 +0000 (04:39 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Jan 2019 04:39:23 +0000 (04:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=193345

Patch by Joseph Pecoraro <pecoraro@apple.com> on 2019-01-10
Reviewed by Simon Fraser.

* MiniBrowser/mac/BrowserWindowController.m:
(-[BrowserWindowController addProtocolIfNecessary:]):
Don't prepend "http://" to "about:" prefixed URLs like "about:blank".

* MiniBrowser/mac/WK1BrowserWindowController.m:
(-[WK1BrowserWindowController fetch:]):
* MiniBrowser/mac/WK2BrowserWindowController.m:
(-[WK2BrowserWindowController fetch:]):
Clean up the code that uses this to match style.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@239856 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/MiniBrowser/mac/BrowserWindowController.m
Tools/MiniBrowser/mac/WK1BrowserWindowController.m
Tools/MiniBrowser/mac/WK2BrowserWindowController.m

index 354d9bd..dcadc79 100644 (file)
@@ -1,5 +1,22 @@
 2019-01-10  Joseph Pecoraro  <pecoraro@apple.com>
 
+        MiniBrowser should be able to navigate to about:blank
+        https://bugs.webkit.org/show_bug.cgi?id=193345
+
+        Reviewed by Simon Fraser.
+
+        * MiniBrowser/mac/BrowserWindowController.m:
+        (-[BrowserWindowController addProtocolIfNecessary:]):
+        Don't prepend "http://" to "about:" prefixed URLs like "about:blank".
+
+        * MiniBrowser/mac/WK1BrowserWindowController.m:
+        (-[WK1BrowserWindowController fetch:]):
+        * MiniBrowser/mac/WK2BrowserWindowController.m:
+        (-[WK2BrowserWindowController fetch:]):
+        Clean up the code that uses this to match style.
+
+2019-01-10  Joseph Pecoraro  <pecoraro@apple.com>
+
         Remove MiniBrowser custom "Process Swap" menu item which does not work
         https://bugs.webkit.org/show_bug.cgi?id=193344
 
index 2817dab..4b99057 100644 (file)
@@ -65,6 +65,9 @@
     if ([address hasPrefix:@"data:"])
         return address;
 
+    if ([address hasPrefix:@"about:"])
+        return address;
+
     return [@"http://" stringByAppendingString:address];
 }
 
index 384c791..62094d9 100644 (file)
@@ -90,7 +90,7 @@
 
 - (IBAction)fetch:(id)sender
 {
-    [urlText setStringValue:[self addProtocolIfNecessary:[urlText stringValue]]];
+    [urlText setStringValue:[self addProtocolIfNecessary:urlText.stringValue]];
     NSURL *url = [NSURL _webkit_URLWithUserTypedString:urlText.stringValue];
     [[_webView mainFrame] loadRequest:[NSURLRequest requestWithURL:url]];
 }
index 3e739d7..3d3b7f3 100644 (file)
@@ -135,9 +135,9 @@ static const int testFooterBannerHeight = 58;
 
 - (IBAction)fetch:(id)sender
 {
-    [urlText setStringValue:[self addProtocolIfNecessary:[urlText stringValue]]];
-
-    [_webView loadRequest:[NSURLRequest requestWithURL:[NSURL _webkit_URLWithUserTypedString:[urlText stringValue]]]];
+    [urlText setStringValue:[self addProtocolIfNecessary:urlText.stringValue]];
+    NSURL *url = [NSURL _webkit_URLWithUserTypedString:urlText.stringValue];
+    [_webView loadRequest:[NSURLRequest requestWithURL:url]];
 }
 
 - (IBAction)setPageScale:(id)sender