Web MIDI: make naming convention be consistent

android_webview/browser/aw_browser_context.cc

 // Copyright (c) 2012 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 "android_webview/browser/aw_browser_context.h"
 
 #include "android_webview/browser/aw_form_database_service.h"
 #include "android_webview/browser/aw_pref_store.h"
 #include "android_webview/browser/aw_quota_manager_bridge.h"
 #include "android_webview/browser/aw_resource_context.h"
@@ skipping 148 matching lines @@
 net::URLRequestContextGetter*
 AwBrowserContext::GetRequestContextForRenderProcess(
     int renderer_child_id) {
   return GetRequestContext();
 }
 
 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,
     const GURL& requesting_frame) {
 }
 
 void AwBrowserContext::RequestProtectedMediaIdentifierPermission(
     int render_process_id,
     int render_view_id,
     int bridge_id,
@@ skipping 52 matching lines @@
 
 void AwBrowserContext::RebuildTable(
     const scoped_refptr<URLEnumerator>& enumerator) {
   // Android WebView rebuilds from WebChromeClient.getVisitedHistory. The client
   // can change in the lifetime of this WebView and may not yet be set here.
   // Therefore this initialization path is not used.
   enumerator->OnComplete(true);
 }
 
 }  // namespace android_webview