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

Unified Diff: chrome/browser/media/midi_permission_infobar_delegate.h

Issue 151343002: Web MIDI: make naming convention be consistent (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review boliu #2 Created 6 years, 11 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: chrome/browser/media/midi_permission_infobar_delegate.h
diff --git a/chrome/browser/media/midi_permission_infobar_delegate.h b/chrome/browser/media/midi_permission_infobar_delegate.h
index eefc3409cf5ff3ffdadd968597861fda8cba90a2..b73fcc766f9e49829ddebea9b65896b81930123d 100644
--- a/chrome/browser/media/midi_permission_infobar_delegate.h
+++ b/chrome/browser/media/midi_permission_infobar_delegate.h
@@ -14,13 +14,10 @@
class PermissionQueueController;
class InfoBarService;
-// TODO(toyoshim): Much more code can be shared with GeolocationInfoBarDelegate.
-// http://crbug.com/266743
-
-// MIDIPermissionInfoBarDelegates are created by the
-// ChromeMIDIPermissionContext to control the display and handling of MIDI
+// MidiPermissionInfoBarDelegates are created by the
+// ChromeMidiPermissionContext to control the display and handling of MIDI
// permission infobars to the user.
-class MIDIPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
+class MidiPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
public:
// Creates a MIDI permission infobar and delegate and adds the infobar to
// |infobar_service|. Returns the infobar if it was successfully added.
@@ -31,12 +28,12 @@ class MIDIPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
const std::string& display_languages);
private:
- MIDIPermissionInfoBarDelegate(PermissionQueueController* controller,
+ MidiPermissionInfoBarDelegate(PermissionQueueController* controller,
const PermissionRequestID& id,
const GURL& requesting_frame,
int contents_unique_id,
const std::string& display_languages);
- virtual ~MIDIPermissionInfoBarDelegate();
+ virtual ~MidiPermissionInfoBarDelegate();
// ConfirmInfoBarDelegate:
virtual void InfoBarDismissed() OVERRIDE;
@@ -59,7 +56,7 @@ class MIDIPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
int contents_unique_id_;
std::string display_languages_;
- DISALLOW_COPY_AND_ASSIGN(MIDIPermissionInfoBarDelegate);
+ DISALLOW_COPY_AND_ASSIGN(MidiPermissionInfoBarDelegate);
};
#endif // CHROME_BROWSER_MEDIA_MIDI_PERMISSION_INFOBAR_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698