Add is_fullscreen to video updates

chrome/browser/chromeos/power/video_activity_notifier.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/chromeos/power/video_activity_notifier.h"
 
 #include "ash/shell.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/power_manager_client.h"
 
 namespace {
 
 // Minimum number of seconds between notifications.
 const int kNotifyIntervalSec = 5;

[Daniel Erat, 2012/08/30 20:33:59]

you should delete this if you're not using it, but see below.

 
 }  // namespace
 
 namespace chromeos {
 
 VideoActivityNotifier::VideoActivityNotifier() {
   ash::Shell::GetInstance()->video_detector()->AddObserver(this);
 }
 
 VideoActivityNotifier::~VideoActivityNotifier() {
   ash::Shell::GetInstance()->video_detector()->RemoveObserver(this);
 }
 
-void VideoActivityNotifier::OnVideoDetected() {
+void VideoActivityNotifier::OnVideoDetected(bool is_fullscreen) {
   base::TimeTicks now = base::TimeTicks::Now();
-  // InSeconds() truncates rather than rounding, so it's fine for this
-  // comparison.
-  if (last_notify_time_.is_null() ||
-      (now - last_notify_time_).InSeconds() >= kNotifyIntervalSec) {
-    DBusThreadManager::Get()->GetPowerManagerClient()->NotifyVideoActivity(now);
-    last_notify_time_ = now;
-  }
+  DBusThreadManager::Get()->GetPowerManagerClient()->

[Daniel Erat, 2012/08/30 20:33:59]

the presence of two rate limits was intentional (well, maybe not intentional,
but known).  within chrome, i wanted to rate-limit how frequently observers get
called; here, i wanted to further rate-limit how frequently we send messages
over d-bus (which is presumably a more-costly action).

i think that the rate limit here should be kept, at the very least, since this
code is better-informed about how frequently we should send d-bus messages than
the upstream code in VideoDetector.

[rharrison, 2012/08/31 18:57:13]

I have reverted this removal.

+      NotifyVideoActivity(now, is_fullscreen);
+  last_notify_time_ = now;
 }
 
 }  // namespace chromeos