Make it possible to type data: URLs into MiniBrowser on Windows
authoraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 10 Jan 2012 21:05:59 +0000 (21:05 +0000)
committeraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 10 Jan 2012 21:05:59 +0000 (21:05 +0000)
Fixes <http://webkit.org/b/75084> Crash when trying to navigate to a data: URL in
MiniBrowser on Windows

Reviewed by Darin Adler.

* MiniBrowser/win/BrowserWindow.cpp:
(BrowserWindow::handleMessage): Instead of checking for an "http://" prefix when deciding
whether to prepend "http://", just look for whether the typed URL contains a colon. If it
does, we assume it already has a scheme and don't modify it.

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

Tools/ChangeLog
Tools/MiniBrowser/win/BrowserWindow.cpp

index 3f42c54..ee22345 100644 (file)
@@ -1,5 +1,19 @@
 2012-01-10  Adam Roben  <aroben@apple.com>
 
+        Make it possible to type data: URLs into MiniBrowser on Windows
+
+        Fixes <http://webkit.org/b/75084> Crash when trying to navigate to a data: URL in
+        MiniBrowser on Windows
+
+        Reviewed by Darin Adler.
+
+        * MiniBrowser/win/BrowserWindow.cpp:
+        (BrowserWindow::handleMessage): Instead of checking for an "http://" prefix when deciding
+        whether to prepend "http://", just look for whether the typed URL contains a colon. If it
+        does, we assume it already has a scheme and don't modify it.
+
+2012-01-10  Adam Roben  <aroben@apple.com>
+
         Make MiniBrowser automatically escape invalid URL characters typed in its URL field
 
         Fixes <http://webkit.org/b/75086> Crash when typing a data: URL containing double-quotes in
index dc43a68..b791d5b 100644 (file)
@@ -251,7 +251,7 @@ bool BrowserWindow::handleMessage(const MSG* message)
 
     std::wstring url(&buffer[0], buffer.size() - 1);
 
-    if (url.find(L"http://"))
+    if (url.find(L":") == std::wstring::npos)
         url = L"http://" + url;
 
     m_browserView.goToURL(url);