Clicking on the HaTS notification now opens the survey dialog

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/command_line.h"
 #include "base/feature_list.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 162 matching lines @@
 
   return true;
 }
 
 // NotificationDelegate override:
 std::string HatsNotificationController::id() const {
   return kDelegateId;
 }
 
 // message_center::NotificationDelegate override:
+void HatsNotificationController::Click() {
+  ButtonClick(0 /* This argument is no-op in the function. */);

[stevenjb, 2016/08/30 15:39:16]

nit: ButtonClick(0 /* unused */);

[malaykeshav, 2016/08/30 18:10:30]

done

+}
+
+// message_center::NotificationDelegate override:
 void HatsNotificationController::ButtonClick(int button_index) {

[stevenjb, 2016/08/30 15:39:16]

nit: int /* button_index */

[malaykeshav, 2016/08/30 18:10:30]

done

   UpdateLastInteractionTime();
 
   // The dialog deletes itslef on close.
   HatsDialog::CreateAndShow(IsGoogleUser(profile_->GetProfileUserName()));
 
   // Remove the notification.
   g_browser_process->notification_ui_manager()->CancelById(
       id(), NotificationUIManager::GetProfileID(profile_));
 }
 
@@ skipping 74 matching lines @@
       GURL(kNotificationOriginUrl), kNotificationId, optional, this);
 }
 
 void HatsNotificationController::UpdateLastInteractionTime() {
   PrefService* pref_service = profile_->GetPrefs();
   pref_service->SetInt64(prefs::kHatsLastInteractionTimestamp,
                          base::Time::Now().ToInternalValue());
 }
 
 }  // namespace chromeos