Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(122)

Unified Diff: chromeos/dbus/power_manager_client.cc

Issue 10905026: Add is_fullscreen to video updates (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chromeos/dbus/power_manager_client.cc
diff --git a/chromeos/dbus/power_manager_client.cc b/chromeos/dbus/power_manager_client.cc
index 5721749325964d0eac4b3ed679dbb42a0a2abc86..f4ab942b68e9988559a4a23bca9d2db00f15aaab 100644
--- a/chromeos/dbus/power_manager_client.cc
+++ b/chromeos/dbus/power_manager_client.cc
@@ -16,6 +16,7 @@
#include "base/timer.h"
#include "chromeos/dbus/power_state_control.pb.h"
#include "chromeos/dbus/power_supply_properties.pb.h"
+#include "chromeos/dbus/video_detect_update.pb.h"
#include "dbus/bus.h"
#include "dbus/message.h"
#include "dbus/object_path.h"
@@ -224,12 +225,18 @@ class PowerManagerClientImpl : public PowerManagerClient {
}
virtual void NotifyVideoActivity(
- const base::TimeTicks& last_activity_time) OVERRIDE {
+ const base::TimeTicks& last_activity_time,
+ bool is_fullscreen) OVERRIDE {
dbus::MethodCall method_call(
power_manager::kPowerManagerInterface,
power_manager::kHandleVideoActivityMethod);
dbus::MessageWriter writer(&method_call);
- writer.AppendInt64(last_activity_time.ToInternalValue());
+
+ VideoDetectUpdate protobuf;
rharrison 2012/08/30 19:16:07 I am going to change the name of this protobuf to
Daniel Erat 2012/08/30 20:33:59 agreed
+ protobuf.set_last_activity_time(last_activity_time.ToInternalValue());
+ protobuf.set_is_fullscreen(is_fullscreen);
+
+ writer.AppendProtoAsArrayOfBytes(protobuf);
power_manager_proxy_->CallMethod(
&method_call,
dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
@@ -589,7 +596,8 @@ class PowerManagerClientStubImpl : public PowerManagerClient {
virtual void NotifyUserActivity(
const base::TimeTicks& last_activity_time) OVERRIDE {}
virtual void NotifyVideoActivity(
- const base::TimeTicks& last_activity_time) OVERRIDE {}
+ const base::TimeTicks& last_activity_time,
+ bool is_fullscreen) OVERRIDE {}
virtual void RequestPowerStateOverrides(
uint32 request_id,
uint32 duration,

Powered by Google App Engine
This is Rietveld 408576698