Add ShouldDisplayOverFullscreen support to web notifications.

chrome/browser/notifications/notification_test_util.h

 // Copyright (c) 2012 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_NOTIFICATIONS_NOTIFICATION_TEST_UTIL_H_
 #define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_TEST_UTIL_H_
 
 #include <set>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "chrome/browser/notifications/notification.h"
 #include "chrome/browser/notifications/notification_ui_manager.h"
 
+class Browser;
 class Profile;
 
 // NotificationDelegate which does nothing, useful for testing when
 // the notification events are not important.
 class MockNotificationDelegate : public NotificationDelegate {
  public:
   explicit MockNotificationDelegate(const std::string& id);
 
   // NotificationDelegate interface.
   std::string id() const override;
@@ skipping 38 matching lines @@
 
  private:
   using NotificationPair = std::pair<Notification, ProfileID>;
   std::vector<NotificationPair> notifications_;
 
   base::Closure notification_added_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(StubNotificationUIManager);
 };
 
+// Helper class that has to be created in the stack to check if the fullscreen
+// setting of a browser is in the desired state.
+class FullscreenStateWaiter {
+ public:
+  explicit FullscreenStateWaiter(Browser* browser, bool desired_state);

[Peter Beverloo, 2016/09/29 21:30:52]

nit: no need to make this explicit (we only do that for single-argument
constructors)

[bmalcolm, 2016/09/29 21:37:13]

Done.

+
+  void Wait();
+
+ private:
+  Browser* browser_;
+  bool desired_state_;
+
+  DISALLOW_COPY_AND_ASSIGN(FullscreenStateWaiter);
+};
+

[Peter Beverloo, 2016/09/29 21:30:52]

micro nit: no double blank lines

[bmalcolm, 2016/09/29 21:37:13]

Done.

+
 #endif  // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_TEST_UTIL_H_