Adds a notification origin URL to solve minor bug

chrome/browser/chromeos/hats/hats_notification_controller.cc

 // Copyright (c) 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/hats/hats_notification_controller.h"
 
 #include "ash/common/system/system_notifier.h"
 #include "base/feature_list.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/hats/hats_dialog.h"
@@ skipping 10 matching lines @@
 #include "grit/ash_strings.h"
 #include "grit/theme_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/message_center/message_center.h"
 #include "ui/message_center/notification_types.h"
 #include "ui/strings/grit/ui_strings.h"
 
 namespace {
 
+const char kNotificationOriginUrl[] = "chrome://hats";
+
 // Returns true if the given |profile| interacted with HaTS by either
 // dismissing the notification or taking the survey within a given threshold
 // days |threshold_days|.
 bool DidShowSurveyToProfileRecently(Profile* profile, int threshold_days) {
   int64_t serialized_timestamp =
       profile->GetPrefs()->GetInt64(prefs::kHatsLastInteractionTimestamp);
 
   base::Time previous_interaction_timestamp =
       base::Time::FromInternalValue(serialized_timestamp);
   return (previous_interaction_timestamp +
@@ skipping 48 matching lines @@
     UpdateLastInteractionTime();
     return;
   }
 
   // Add self as an observer to be notified when an internet connection is
   // available.
   network_portal_detector::GetInstance()->AddAndFireObserver(this);
 }
 
 // static
-// TODO(malaykeshav): Add check for @google accounts.
 bool HatsNotificationController::ShouldShowSurveyToProfile(Profile* profile) {
   // Do not show the survey if the HaTS feature is disabled for the device. This
   // flag is controlled by finch and is enabled only when the device has been
   // selected for the survey.
   if (!base::FeatureList::IsEnabled(features::kHappininessTrackingSystem))
     return false;
 
   // Do not show survey if this is a guest session.
   if (profile->IsGuestSession())
     return false;
@@ skipping 68 matching lines @@
   return new Notification(
       message_center::NOTIFICATION_TYPE_SIMPLE,
       l10n_util::GetStringUTF16(IDS_ASH_HATS_NOTIFICATION_TITLE),
       l10n_util::GetStringUTF16(IDS_ASH_HATS_NOTIFICATION_BODY),
       // TODO(malaykeshav): Change this to actual HaTS icon.
       ui::ResourceBundle::GetSharedInstance().GetImageNamed(
           IDR_SCREENSHOT_NOTIFICATION_ICON),
       message_center::NotifierId(message_center::NotifierId::SYSTEM_COMPONENT,
                                  ash::system_notifier::kNotifierHats),
       l10n_util::GetStringUTF16(IDS_MESSAGE_CENTER_NOTIFIER_HATS_NAME),
-      GURL() /* Send an empty invalid url */, kNotificationId, optional, this);
+      GURL(kNotificationOriginUrl), kNotificationId, optional, this);
 }
 
 void HatsNotificationController::UpdateLastInteractionTime() {
   PrefService* pref_service = profile_->GetPrefs();
   pref_service->SetInt64(prefs::kHatsLastInteractionTimestamp,
                          base::Time::Now().ToInternalValue());
 }
 
 }  // namespace chromeos