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

Unified Diff: content/browser/renderer_host/device_orientation_browser_message_filter.h

Issue 20707002: Implement Device Orientation using shared memory in content/renderer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 5 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: content/browser/renderer_host/device_orientation_browser_message_filter.h
diff --git a/content/browser/renderer_host/device_orientation_browser_message_filter.h b/content/browser/renderer_host/device_orientation_browser_message_filter.h
new file mode 100644
index 0000000000000000000000000000000000000000..575219a356c20812b08b1b4657e5ba0ec3fc8d0e
--- /dev/null
+++ b/content/browser/renderer_host/device_orientation_browser_message_filter.h
@@ -0,0 +1,39 @@
+ // Copyright 2013 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 CONTENT_BROWSER_RENDERER_HOST_DEVICE_ORIENTATION_BROWSER_MESSAGE_FILTER_H_
+ #define CONTENT_BROWSER_RENDERER_HOST_DEVICE_ORIENTATION_BROWSER_MESSAGE_FILTER_H_
darin (slow to review) 2013/07/31 19:27:28 is there a reason why this file doesn't exist in c
timvolodine 2013/08/01 15:31:36 AFAIK there is no reason, so moved it there. I nee
+
+ #include "base/compiler_specific.h"
+ #include "base/memory/shared_memory.h"
darin (slow to review) 2013/07/31 19:27:28 nit: looks like this include is not needed
timvolodine 2013/08/01 15:31:36 Done.
+ #include "content/public/browser/browser_message_filter.h"
+
+ namespace content {
+
+ class DeviceMotionService;
+ class RenderProcessHost;
+
+ class DeviceOrientationBrowserMessageFilter : public BrowserMessageFilter {
darin (slow to review) 2013/07/31 19:27:28 I think this class should be named DeviceOrientati
timvolodine 2013/08/01 15:31:36 Done.
+ public:
+ DeviceOrientationBrowserMessageFilter();
+
+ // BrowserMessageFilter implementation.
+ virtual bool OnMessageReceived(const IPC::Message& message,
+ bool* message_was_ok) OVERRIDE;
+
+ private:
+ virtual ~DeviceOrientationBrowserMessageFilter();
+
+ void OnDeviceOrientationStartPolling();
+ void OnDeviceOrientationStopPolling();
+ void DidStartDeviceOrientationPolling();
+
+ bool is_started_;
+
+ DISALLOW_COPY_AND_ASSIGN(DeviceOrientationBrowserMessageFilter);
+ };
+
+ } // namespace content
+
+ #endif // CONTENT_BROWSER_RENDERER_HOST_DEVICE_ORIENTATION_BROWSER_MESSAGE_FILTER_H_

Powered by Google App Engine
This is Rietveld 408576698