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

Unified Diff: content/browser/vibration_browsertest.cc

Issue 2859343003: Enable overriding interface binders for any services running in current process. (Closed)
Patch Set: Created 3 years, 7 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
« no previous file with comments | « no previous file | content/test/BUILD.gn » ('j') | services/service_manager/public/cpp/service_context.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/vibration_browsertest.cc
diff --git a/content/browser/vibration_browsertest.cc b/content/browser/vibration_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..7e488766ff5ccfd82b0cfbbca23cca01c00dd329
--- /dev/null
+++ b/content/browser/vibration_browsertest.cc
@@ -0,0 +1,90 @@
+// Copyright 2017 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 <utility>
+
+#include "base/bind.h"
+#include "base/macros.h"
+#include "base/run_loop.h"
+#include "base/strings/string_number_conversions.h"
+#include "content/public/browser/render_frame_host.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/test/browser_test_utils.h"
+#include "content/public/test/content_browser_test.h"
+#include "content/public/test/content_browser_test_utils.h"
+#include "content/shell/browser/shell.h"
+#include "mojo/public/cpp/bindings/binding.h"
+#include "services/device/public/interfaces/constants.mojom.h"
+#include "services/device/public/interfaces/vibration_manager.mojom.h"
+#include "services/service_manager/public/cpp/service_context.h"
+
+namespace content {
+
+namespace {
+
+class VibrationTest : public ContentBrowserTest,
+ public device::mojom::VibrationManager {
+ public:
+ VibrationTest() : binding_(this){};
+
+ void SetUpOnMainThread() override {
+ // Because Device Service also runs in this process(browser process), we can
+ // set our binder to intercept requests for VibrationManager interface to
+ // it.
+ service_manager::ServiceContext::TestApi::SetGlobalBinder(
+ service_manager::Identity(device::mojom::kServiceName),
+ device::mojom::VibrationManager::Name_,
+ base::Bind(&VibrationTest::BindVibrationManager,
+ base::Unretained(this)));
+ }
+
+ void BindVibrationManager(mojo::ScopedMessagePipeHandle handle) {
+ binding_.Bind(std::move(handle));
+ }
+
+ protected:
+ bool TriggerVibrate(int duration, base::Closure vibrate_done) {
+ vibrate_done_ = vibrate_done;
+
+ bool result;
+ RenderFrameHost* frame = shell()->web_contents()->GetMainFrame();
+ std::string script = "domAutomationController.send(navigator.vibrate(" +
+ base::IntToString(duration) + "))";
+ EXPECT_TRUE(ExecuteScriptAndExtractBool(frame, script, &result));
+ return result;
+ }
+
+ int64_t vibrate_milliseconds() { return vibrate_milliseconds_; }
+
+ private:
+ // device::mojom::VibrationManager:
+ void Vibrate(int64_t milliseconds, const VibrateCallback& callback) override {
+ vibrate_milliseconds_ = milliseconds;
+ callback.Run();
+ // Quit the run loop.
+ vibrate_done_.Run();
+ }
+ void Cancel(const CancelCallback& callback) override { callback.Run(); }
+
+ int64_t vibrate_milliseconds_ = -1;
+ base::Closure vibrate_done_;
+ mojo::Binding<device::mojom::VibrationManager> binding_;
+
+ DISALLOW_COPY_AND_ASSIGN(VibrationTest);
+};
+
+IN_PROC_BROWSER_TEST_F(VibrationTest, Vibrate) {
+ ASSERT_EQ(-1, vibrate_milliseconds());
+
+ ASSERT_TRUE(NavigateToURL(shell(), GetTestUrl(".", "simple_page.html")));
+ base::RunLoop run_loop;
+ ASSERT_TRUE(TriggerVibrate(1234, run_loop.QuitClosure()));
+ run_loop.Run();
+
+ ASSERT_EQ(1234, vibrate_milliseconds());
+}
+
+} // namespace
+
+} // namespace content
« no previous file with comments | « no previous file | content/test/BUILD.gn » ('j') | services/service_manager/public/cpp/service_context.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698