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_VIBRATION_VIBRATION_MESSAGE_FILTER_H_

[bulach, 2013/06/20 14:21:04]

nit: CONTENT_BROWSER_...

[Michael van Ouwerkerk, 2013/06/20 17:34:09]

Done.

+#define CONTENT_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(VibrationService* vibration_service);

[bulach, 2013/06/20 14:21:04]

nit: explicit. also, since this is taking ownership,
scoped_ptr<VibrationService> vibration_service

[Michael van Ouwerkerk, 2013/06/20 17:34:09]

Done.

+
+  // BrowserMessageFilter implementation.
+  virtual bool OnMessageReceived(const IPC::Message& message,
+                                 bool* message_was_ok) OVERRIDE;
+
+ private:
+  virtual ~VibrationMessageFilter();
+
+  void OnVibrate(unsigned milliseconds);

[bulach, 2013/06/20 14:21:04]

int?

[Michael van Ouwerkerk, 2013/06/20 17:34:09]

Like in the other comment... could you explain why? It is unsigned all the way
through.

+  void OnCancelVibration();
+
+  scoped_ptr<VibrationService> vibration_service_;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_VIBRATION_VIBRATION_MESSAGE_FILTER_H_