os-settings: check URL origin for link capture

Let browser_navigator check for OS Settings origin when it decides to
open OS Settings in a standalone window.

Before this CL, browser_navigator ignores the scheme and checks for
os-settings host only. Thus URLs like http://os-settings/ will open a
standalone window (with OS Settings icon). This is misleading.

Fixed: 1114023
Change-Id: I8a8f5d21e3f3d6aae0d4baea9194ac64d065fcd2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2342232Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Jiewei Qian  <qjw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#797513}

chrome/browser/ui/browser_navigator.cc

@@ -529,7 +529,8 @@ void Navigate(NavigateParams* params) {
 #if defined(OS_CHROMEOS)
   if (source_browser) {
     // Open OS settings in PWA, even when user types in URL bar.
-    if (params->url.host() == chrome::kChromeUIOSSettingsHost &&
+    if (params->url.GetOrigin() ==
+            GURL(chrome::kChromeUIOSSettingsURL).GetOrigin() &&
         !allow_os_settings_in_tab) {
       chrome::SettingsWindowManager* settings_window_manager =
           chrome::SettingsWindowManager::GetInstance();