Push API: Don't require notification if tab is visible.

chrome/browser/ui/android/tab_model/tab_model.cc

 // 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.
 
 #include "chrome/browser/ui/android/tab_model/tab_model.h"
 
 #include "base/logging.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sync/glue/synced_window_delegate_android.h"
 #include "chrome/browser/ui/toolbar/toolbar_model_impl.h"
 #include "content/public/browser/notification_service.h"
 
 using content::NotificationService;
 
+static int INVALID_TAB_INDEX = -1;

[Michael van Ouwerkerk, 2015/01/21 19:13:22]

Similar comment as in TabList.java?

[johnme, 2015/01/21 19:32:19]

Done.

+
 TabModel::TabModel(Profile* profile)
   : profile_(profile),
     synced_window_delegate_(
         new browser_sync::SyncedWindowDelegateAndroid(this)) {
 
   if (profile) {
     // A normal Profile creates an OTR profile if it does not exist when
     // GetOffTheRecordProfile() is called, so we guard it with
     // HasOffTheRecordProfile(). An OTR profile returns itself when you call
     // GetOffTheRecordProfile().
@@ skipping 24 matching lines @@
 }
 
 browser_sync::SyncedWindowDelegate* TabModel::GetSyncedWindowDelegate() const {
   return synced_window_delegate_.get();
 }
 
 SessionID::id_type TabModel::GetSessionId() const {
   return session_id_.id();
 }
 
+content::WebContents* TabModel::GetActiveWebContents() const {
+  int active_index = GetActiveIndex();
+  if (active_index == INVALID_TAB_INDEX)
+    return nullptr;
+  return GetWebContentsAt(active_index);
+}
+
 void TabModel::BroadcastSessionRestoreComplete() {
   if (profile_) {
     NotificationService::current()->Notify(
         chrome::NOTIFICATION_SESSION_RESTORE_COMPLETE,
         content::Source<Profile>(profile_),
         NotificationService::NoDetails());
   } else {
     // TODO(nyquist): Uncomment this once downstream Android uses new
     // constructor that takes a Profile* argument. See crbug.com/159704.
     // NOTREACHED();
@@ skipping 17 matching lines @@
       if (is_off_the_record_) {
         Profile* profile = content::Source<Profile>(source).ptr();
         if (profile && profile->IsOffTheRecord())
           profile_ = profile;
       }
       break;
     default:
       NOTREACHED();
   }
 }