Web MIDI: make naming convention be consistent

android_webview/browser/aw_browser_context.cc

Index: android_webview/browser/aw_browser_context.cc
diff --git a/android_webview/browser/aw_browser_context.cc b/android_webview/browser/aw_browser_context.cc
index 109c3d7cee4a77f679c425c089563a1e6a8c086f..8292a16a6719dbbba764bcaae1d9a1866470b504 100644
--- a/android_webview/browser/aw_browser_context.cc
+++ b/android_webview/browser/aw_browser_context.cc
@@ -166,17 +166,17 @@ net::URLRequestContextGetter* AwBrowserContext::GetMediaRequestContext() {
   return GetRequestContext();
 }
 
-void AwBrowserContext::RequestMIDISysExPermission(
+void AwBrowserContext::RequestMidiSysExPermission(
       int render_process_id,
       int render_view_id,
       int bridge_id,
       const GURL& requesting_frame,
-      const MIDISysExPermissionCallback& callback) {
-  // TODO(toyoshim): Android is not supported yet.
+      const MidiSysExPermissionCallback& callback) {
+  // TODO(crbug.com/339767): Android is not supported yet.

[boliu, 2014/01/31 15:51:33]

nit: This is unconventional. Why not just put the bug in the comment after the
TODO? Also this is android webview, not chrome on android the browser.

[scherkus (not reviewing), 2014/01/31 20:48:54]

+1 -- keep it:

TODO(username): Summary of what needs to be fixed http://crbug.com/339767

[Takashi Toyoshima, 2014/02/03 04:24:53]

Thanks. I update the comment to say it's WebView.
And, keep TODO(username): comment style.

   callback.Run(false);
 }
 
-void AwBrowserContext::CancelMIDISysExPermissionRequest(
+void AwBrowserContext::CancelMidiSysExPermissionRequest(
     int render_process_id,
     int render_view_id,
     int bridge_id,