Add ShouldDisplayOverFullscreen support to web notifications.

chrome/browser/notifications/web_notification_delegate.cc

+// Copyright 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/notifications/web_notification_delegate.h"
+
+#include "chrome/browser/notifications/notification_common.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_window.h"
+#include "chrome/browser/ui/exclusive_access/exclusive_access_context.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "content/public/browser/web_contents.h"
+
+WebNotificationDelegate::WebNotificationDelegate(
+    content::BrowserContext* browser_context,
+    const std::string& notification_id,
+    const GURL& origin)
+    : browser_context_(browser_context),
+      notification_id_(notification_id),
+      origin_(origin) {}
+
+WebNotificationDelegate::~WebNotificationDelegate() {}
+
+std::string WebNotificationDelegate::id() const {
+  return notification_id_;
+}
+
+void WebNotificationDelegate::SettingsClick() {
+  NotificationCommon::OpenNotificationSettings(browser_context_);
+}
+
+bool WebNotificationDelegate::ShouldDisplaySettingsButton() {
+  return true;
+}
+
+bool WebNotificationDelegate::ShouldDisplayOverFullscreen() const {
+  Profile* profile = Profile::FromBrowserContext(browser_context_);

[Peter Beverloo, 2016/09/28 12:23:13]

Oops, sorry that I missed this - this code can't be used on Android, because
Android uses it own (Java) variants of the TabStripModel.

But that's OK, because the system UI there will decide whether to display the
notification or not, so we can just hoist lines 39-63 in an #if
!defined(OS_ANDROID) block.

[bmalcolm, 2016/09/28 18:30:09]

Done.

+  DCHECK(profile);
+  // Check to see if this notification comes from a webpage that is displaying
+  // fullscreen content.
+  for (auto* browser : *BrowserList::GetInstance()) {
+    // Only consider the browsers for the profile that created the notification
+    if (browser->profile() != profile)
+      continue;
+
+    const content::WebContents* active_contents =
+        browser->tab_strip_model()->GetActiveWebContents();
+    if (!active_contents)
+      continue;
+
+    // Check to see if
+    //  (a) the active tab in the browser shares its origin with the
+    //      notification.
+    //  (b) the browser is fullscreen
+    //  (c) the browser has focus.
+    if (active_contents->GetURL().GetOrigin() == origin_ &&
+        browser->exclusive_access_manager()->context()->IsFullscreen() &&
+        browser->window()->IsActive()) {
+      return true;
+    }
+  }
+
+  return false;
+}