Added in-session captive portal notification.

chrome/browser/chromeos/net/network_portal_notification_controller.h

+// Copyright 2014 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_NETWORK_PORTAL_NOTIFICATION_CONTROLLER_H_
+#define CHROME_BROWSER_CHROMEOS_NET_NETWORK_PORTAL_NOTIFICATION_CONTROLLER_H_
+
+#include <string>
+
+#include "base/basictypes.h"

[oshima, 2014/02/20 19:00:54]

#include "base/compiler_specific.h"

[ygorshenin1, 2014/02/20 19:18:45]

I'm not using OVERRIDE here, why do I need this header?

On 2014/02/20 19:00:54, oshima wrote:

[oshima, 2014/02/20 20:26:50]

Sorry what I really meant was "base/macros.h"

[ygorshenin1, 2014/02/21 09:04:56]

Done.

+#include "chrome/browser/chromeos/net/network_portal_detector.h"
+
+namespace chromeos {
+
+class NetworkState;
+
+class NetworkPortalNotificationController {
+ public:
+  static const char kNotificationId[];
+
+  NetworkPortalNotificationController();
+  virtual ~NetworkPortalNotificationController();
+
+  void OnPortalDetectionCompleted(
+      const NetworkState* network,
+      const NetworkPortalDetector::CaptivePortalState& state);
+
+ private:
+  // Last network path for which notification was displayed.
+  std::string last_network_path_;
+
+  DISALLOW_COPY_AND_ASSIGN(NetworkPortalNotificationController);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_NET_NETWORK_PORTAL_NOTIFICATION_CONTROLLER_H_