Vibration API: plumbing from Blink to Java.

content/browser/vibration/vibration_message_filter.h

+// Copyright (c) 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_VIBRATION_VIBRATION_MESSAGE_FILTER_H_
+#define CONTENT_BROWSER_VIBRATION_VIBRATION_MESSAGE_FILTER_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "content/browser/vibration/vibration_service.h"
+#include "content/public/browser/browser_message_filter.h"
+
+namespace content {
+
+class VibrationMessageFilter : public BrowserMessageFilter {
+ public:
+  VibrationMessageFilter();
+
+  // BrowserMessageFilter implementation.
+  virtual bool OnMessageReceived(const IPC::Message& message,

[jam, 2013/07/01 18:14:40]

nit: just hide this in private

[Michael van Ouwerkerk, 2013/07/02 17:52:39]

I don't mind either way, but could you help me understand why? OnMessageReceived
is public in BrowserMessageFilter and all subclasses I looked at. Are they all
wrong?

[jam, 2013/07/02 22:11:15]

in general better to hide members if they don't need to be public

[Michael van Ouwerkerk, 2013/07/03 13:46:33]

Done.

+                                 bool* message_was_ok) OVERRIDE;
+
+ private:
+  virtual ~VibrationMessageFilter();
+
+  void OnVibrate(int64 milliseconds);
+  void OnCancelVibration();
+
+  scoped_ptr<VibrationService> vibration_service_;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_VIBRATION_VIBRATION_MESSAGE_FILTER_H_