Web MIDI: use midi_service.mojom for media::midi::Result

components/test_runner/test_runner.cc

Index: components/test_runner/test_runner.cc
diff --git a/components/test_runner/test_runner.cc b/components/test_runner/test_runner.cc
index 86185b48a8917e53e83714249e615ee18fc4d34a..461e9516091edaa26d3e2689f95b79ec5905082f 100644
--- a/components/test_runner/test_runner.cc
+++ b/components/test_runner/test_runner.cc
@@ -1349,8 +1349,11 @@ void TestRunnerBindings::SetPOSIXLocale(const std::string& locale) {
 }
 
 void TestRunnerBindings::SetMIDIAccessorResult(bool result) {
-  if (runner_)
-    runner_->SetMIDIAccessorResult(result);
+  if (runner_) {
+    runner_->SetMIDIAccessorResult(
+        result ? midi::mojom::Result::OK

[Reilly Grant (use Gerrit), 2016/10/13 21:34:36]

It's completely reasonable to add "using midi::mojom::Result" to the top of a
C++ file like this, though "Result" may be too generic to do that without
conflicts.

[Takashi Toyoshima, 2016/10/14 08:45:28]

Yeah, probably, I want not to use "using midi::mojom::Result" in common files
like test_runner, and use it in midi specific files, like midi_host.cc,
midi_message_filter.cc, and so on.

+               : midi::mojom::Result::INITIALIZATION_ERROR);
+  }
 }
 
 void TestRunnerBindings::SimulateWebNotificationClick(const std::string& title,
@@ -1645,7 +1648,7 @@ void TestRunner::Reset() {
   dump_back_forward_list_ = false;
   test_repaint_ = false;
   sweep_horizontally_ = false;
-  midi_accessor_result_ = true;
+  midi_accessor_result_ = midi::mojom::Result::OK;
   has_custom_text_output_ = false;
   custom_text_output_.clear();
 
@@ -1942,7 +1945,7 @@ bool TestRunner::shouldDumpNavigationPolicy() const {
   return layout_test_runtime_flags_.dump_navigation_policy();
 }
 
-bool TestRunner::midiAccessorResult() {
+midi::mojom::Result TestRunner::midiAccessorResult() {
   return midi_accessor_result_;
 }
 
@@ -2681,7 +2684,7 @@ void TestRunner::SetPOSIXLocale(const std::string& locale) {
   delegate_->SetLocale(locale);
 }
 
-void TestRunner::SetMIDIAccessorResult(bool result) {
+void TestRunner::SetMIDIAccessorResult(midi::mojom::Result result) {
   midi_accessor_result_ = result;
 }