[CrOS Tether] Open the Tether settings page when a tether notification body is clicked.

chrome/browser/chromeos/net/tether_notification_presenter.h

 // Copyright 2017 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_NET_TETHER_NOTIFICATION_PRESENTER_H_
 #define CHROME_BROWSER_CHROMEOS_NET_TETHER_NOTIFICATION_PRESENTER_H_
 
 #include <memory>
 #include <string>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/strings/string16.h"
 #include "chromeos/components/tether/notification_presenter.h"
 #include "chromeos/network/network_connect.h"
 #include "components/cryptauth/remote_device.h"
 #include "ui/message_center/message_center_observer.h"
 #include "ui/message_center/notification.h"
 
+class Profile;
+
 namespace message_center {
 class MessageCenter;
 class Notification;
 }  // namespace message_center
 
 namespace chromeos {
 
 namespace tether {
 
 // Produces notifications associated with CrOS tether network events and alerts
 // observers about interactions with those notifications.
 class TetherNotificationPresenter
     : public NotificationPresenter,
       public message_center::MessageCenterObserver {
  public:
-  // Caller must ensure that |message_center| amd |network_connect| outlive
-  // this instance.
-  TetherNotificationPresenter(message_center::MessageCenter* message_center,
+  // Caller must ensure that |profile|, |message_center|, and |network_connect|
+  // outlive this instance.
+  TetherNotificationPresenter(Profile* profile,
+                              message_center::MessageCenter* message_center,
                               NetworkConnect* network_connect);
   ~TetherNotificationPresenter() override;
 
   // NotificationPresenter:
   void NotifyPotentialHotspotNearby(
       const cryptauth::RemoteDevice& remote_device) override;
   void NotifyMultiplePotentialHotspotsNearby() override;
   void RemovePotentialHotspotNotification() override;
   void NotifyConnectionToHostFailed() override;
   void RemoveConnectionToHostFailedNotification() override;
 
   // message_center::MessageCenterObserver:
   void OnNotificationClicked(const std::string& notification_id) override;
   void OnNotificationButtonClicked(const std::string& notification_id,
                                    int button_index) override;
 
  private:
-  friend class TetherNotificationPresenterTest;
-
   static const char kTetherNotifierId[];
   static const char kPotentialHotspotNotificationId[];
   static const char kActiveHostNotificationId[];
 
   static std::unique_ptr<message_center::Notification> CreateNotification(
       const std::string& id,
       const base::string16& title,
       const base::string16& message);
   static std::unique_ptr<message_center::Notification> CreateNotification(
       const std::string& id,
       const base::string16& title,
       const base::string16& message,
       const message_center::RichNotificationData rich_notification_data);
 
+  friend class TetherNotificationPresenterTest;
+
+  class SettingsUiDelegate {
+   public:
+    SettingsUiDelegate();
+    virtual ~SettingsUiDelegate();

[stevenjb, 2017/05/03 23:07:44]

blank line and comment for ShowSettingsSubPageForProfile

[Kyle Horimoto, 2017/05/03 23:25:48]

Done.

+    virtual void ShowSettingsSubPageForProfile(Profile* profile,
+                                               const std::string& sub_page);

[stevenjb, 2017/05/03 23:07:44]

This should be a pure virtual. Create a local Impl class in the .cc file. Also
make the destructor protected.

[Kyle Horimoto, 2017/05/03 23:25:48]

Done, but I made the destructor public. Otherwise, I got a compiler error about
how unique_ptr can't delete it.

[stevenjb, 2017/05/03 23:42:37]

Acknowledged.

+  };
+
+  void SetSettingsUiDelegateForTesting(
+      std::unique_ptr<SettingsUiDelegate> settings_ui_delegate);

[stevenjb, 2017/05/03 23:07:44]

blank line

[Kyle Horimoto, 2017/05/03 23:25:48]

Done.

   void ShowNotification(
       std::unique_ptr<message_center::Notification> notification);
 
+  Profile* profile_;
   message_center::MessageCenter* message_center_;
   NetworkConnect* network_connect_;
 
+  std::unique_ptr<SettingsUiDelegate> settings_ui_delegate_;
+
   cryptauth::RemoteDevice hotspot_nearby_device_;
   base::WeakPtrFactory<TetherNotificationPresenter> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TetherNotificationPresenter);
 };
 
 }  // namespace tether
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_NET_TETHER_NOTIFICATION_PRESENTER_H_