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

Unified Diff: components/test_runner/test_runner.h

Issue 2404443002: Web MIDI: use midi_service.mojom for media::midi::Result (Closed)
Patch Set: yhirano@ review Created 4 years, 2 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: components/test_runner/test_runner.h
diff --git a/components/test_runner/test_runner.h b/components/test_runner/test_runner.h
index ac2ebd935dc57b3d32cf7d54b82bfe1b11ac8f5f..8bd92787b962b7e99ff168e3d77dde37a673bea4 100644
--- a/components/test_runner/test_runner.h
+++ b/components/test_runner/test_runner.h
@@ -18,6 +18,7 @@
#include "components/test_runner/layout_test_runtime_flags.h"
#include "components/test_runner/test_runner_export.h"
#include "components/test_runner/web_test_runner.h"
+#include "media/midi/midi_service.mojom.h"
#include "third_party/WebKit/public/platform/WebEffectiveConnectionType.h"
#include "third_party/WebKit/public/platform/WebImage.h"
#include "v8/include/v8.h"
@@ -161,7 +162,7 @@ class TestRunner : public WebTestRunner {
void setDragImage(const blink::WebImage& drag_image);
bool shouldDumpNavigationPolicy() const;
- bool midiAccessorResult();
+ midi::mojom::Result midiAccessorResult();
// Methods used by MockColorChooser:
void DidOpenChooser();
@@ -526,7 +527,7 @@ class TestRunner : public WebTestRunner {
void SetPOSIXLocale(const std::string& locale);
// MIDI function to control permission handling.
- void SetMIDIAccessorResult(bool result);
+ void SetMIDIAccessorResult(midi::mojom::Result result);
// Simulates a click on a Web Notification.
void SimulateWebNotificationClick(const std::string& title, int action_index);
@@ -601,8 +602,8 @@ class TestRunner : public WebTestRunner {
// a series of 1px-wide, view-tall paints across the width of the view.
bool sweep_horizontally_;
- // If false, MockWebMIDIAccessor fails on startSession() for testing.
- bool midi_accessor_result_;
+ // startSession() result of MockWebMIDIAccessor for testing.
+ midi::mojom::Result midi_accessor_result_;
bool has_custom_text_output_;
std::string custom_text_output_;

Powered by Google App Engine
This is Rietveld 408576698