[WebPush] Fix silent push exception for Android Messages for Web

The ShouldSkipUserVisibleOnlyRequirements method compares messages
app url against the origin. This breaks the whitelisting since the
app is now hosted at non-root path. Fixed this by comparing the
origin of the app url.

Bug: 936285
Change-Id: I5f71017148ec34cfbb45ae0c74aeffff96b88614
Reviewed-on: https://chromium-review.googlesource.com/c/1490755Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: Jeremy Klein <jlklein@chromium.org>
Commit-Queue: Azeem Arshad <azeemarshad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636108}

chrome/browser/push_messaging/push_messaging_notification_manager.cc

@@ -339,7 +339,7 @@ bool PushMessagingNotificationManager::ShouldSkipUserVisibleOnlyRequirements(
   if (!app_url)
     app_url = chromeos::android_sms::GetAndroidMessagesURL();
 
-  if (!origin.EqualsIgnoringRef(*app_url))
+  if (!origin.EqualsIgnoringRef(app_url->GetOrigin()))
     return false;
 
   return true;

chrome/browser/push_messaging/push_messaging_notification_manager_unittest.cc

@@ -56,11 +56,11 @@ TEST_F(PushMessagingNotificationManagerTest, IsTabVisibleViewSource) {
 #if defined(OS_CHROMEOS)
 TEST_F(PushMessagingNotificationManagerTest,
        SkipEnforceUserVisibleOnlyRequirementsForAndroidMessages) {
-  GURL origin("https://example.com");
+  GURL app_url("https://example.com/test/");
   chromeos::android_sms::FakeAndroidSmsAppManager*
       fake_android_sms_app_manager =
           new chromeos::android_sms::FakeAndroidSmsAppManager();
-  fake_android_sms_app_manager->SetInstalledAppUrl(origin);
+  fake_android_sms_app_manager->SetInstalledAppUrl(app_url);
 
   chromeos::multidevice_setup::FakeMultiDeviceSetupClient*
       fake_multidevice_setup_client =
@@ -75,7 +75,7 @@ TEST_F(PushMessagingNotificationManagerTest,
 
   bool was_called = false;
   manager.EnforceUserVisibleOnlyRequirements(
-      origin, 0l,
+      app_url.GetOrigin(), 0l,
       base::BindRepeating([](bool* was_called) { *was_called = true; },
                           &was_called));
   EXPECT_TRUE(was_called);