[MIDI] Add a permission bubble client for MIDI system control.

chrome/browser/media/chrome_midi_permission_context.cc

 // Copyright 2013 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 "chrome/browser/media/chrome_midi_permission_context.h"
 
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/content_settings/host_content_settings_map.h"
 #include "chrome/browser/content_settings/permission_queue_controller.h"
 #include "chrome/browser/content_settings/permission_request_id.h"
 #include "chrome/browser/content_settings/tab_specific_content_settings.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/tab_contents/tab_util.h"
+#include "chrome/browser/ui/website_settings/permission_bubble_manager.h"
+#include "chrome/browser/ui/website_settings/permission_bubble_request.h"
+#include "chrome/common/pref_names.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents.h"
+#include "grit/generated_resources.h"
+#include "net/base/net_util.h"
+#include "ui/base/l10n/l10n_util.h"
+
+class MidiPermissionRequest : public PermissionBubbleRequest {
+ public:
+  MidiPermissionRequest(
+      ChromeMidiPermissionContext* context,
+      const PermissionRequestID& id,
+      const GURL& requesting_frame,
+      const std::string& display_languages,
+      const content::BrowserContext::MidiSysExPermissionCallback& callback);
+  virtual ~MidiPermissionRequest();
+
+  // PermissionBubbleDelegate:
+  virtual base::string16 GetMessageText() const OVERRIDE;
+  virtual base::string16 GetMessageTextFragment() const OVERRIDE;
+  virtual base::string16 GetAlternateAcceptButtonText() const OVERRIDE;
+  virtual base::string16 GetAlternateDenyButtonText() const OVERRIDE;
+  virtual void PermissionGranted() OVERRIDE;
+  virtual void PermissionDenied() OVERRIDE;
+  virtual void Cancelled() OVERRIDE;
+  virtual void RequestFinished() OVERRIDE;
+
+ private:
+  ChromeMidiPermissionContext* context_;

[Ami GONE FROM CHROMIUM, 2014/02/06 19:51:17]

This is a raw pointer; what guarantees it remains alive for the lifetime of this
object?

[Greg Billock, 2014/02/07 18:20:55]

The permission context is a profile keyed service, which will outlive any
windows (and tabs) associated with that profile. The infobar gets created with
an unretained raw pointer as well.

+  const PermissionRequestID id_;
+  GURL requesting_frame_;
+  int contents_unique_id_;

[Ami GONE FROM CHROMIUM, 2014/02/06 19:51:17]

unused
(why didn't your compiler tell you this?)

[Greg Billock, 2014/02/07 18:20:55]

not sure. Removing.

+  std::string display_languages_;
+  const content::BrowserContext::MidiSysExPermissionCallback& callback_;
+
+  DISALLOW_COPY_AND_ASSIGN(MidiPermissionRequest);
+};
+
+MidiPermissionRequest::MidiPermissionRequest(
+    ChromeMidiPermissionContext* context,
+    const PermissionRequestID& id,
+    const GURL& requesting_frame,
+    const std::string& display_languages,
+    const content::BrowserContext::MidiSysExPermissionCallback& callback)
+    : context_(context),
+      id_(id),
+      requesting_frame_(requesting_frame),
+      display_languages_(display_languages),
+      callback_(callback) {}
+
+MidiPermissionRequest::~MidiPermissionRequest() {}
+
+base::string16 MidiPermissionRequest::GetMessageText() const {
+  return l10n_util::GetStringFUTF16(
+      IDS_MIDI_SYSEX_INFOBAR_QUESTION,
+      net::FormatUrl(requesting_frame_.GetOrigin(), display_languages_));
+}
+
+base::string16 MidiPermissionRequest::GetMessageTextFragment() const {
+  return l10n_util::GetStringUTF16(IDS_MIDI_SYSEX_PERMISSION_FRAGMENT);
+}
+
+base::string16 MidiPermissionRequest::GetAlternateAcceptButtonText() const {
+  return l10n_util::GetStringUTF16(IDS_MIDI_SYSEX_ALLOW_BUTTON);
+}
+
+base::string16 MidiPermissionRequest::GetAlternateDenyButtonText() const {
+  return l10n_util::GetStringUTF16(IDS_MIDI_SYSEX_DENY_BUTTON);

[Ami GONE FROM CHROMIUM, 2014/02/06 19:51:17]

This seems pretty stock.  I wonder if PermissionBubbleDelegate ought to have a
ctor that takes these strings and implements these methods so each impl doesn't
have to repeat them.

[Greg Billock, 2014/02/07 18:20:55]

That's an option. The interface is pure right now, which is always nice... The
strings can be request-context-sensitive, but so far I haven't seen any that
aren't known at construct-time.

[Ami GONE FROM CHROMIUM, 2014/02/07 18:25:34]

Yeah, nice for job security for the guy who re-implements the pure interface
every week! :-P

[Greg Billock, 2014/02/10 23:57:10]

:-) We have eight of them. I guess that's perhaps cumulative 5 minutes of coding
time difference, but I'm not sure which direction... <troll :-)>

+}
+
+void MidiPermissionRequest::PermissionGranted() {
+  context_->NotifyPermissionSet(id_, requesting_frame_, callback_, true);
+}
+
+void MidiPermissionRequest::PermissionDenied() {
+  context_->NotifyPermissionSet(id_, requesting_frame_, callback_, false);
+}
+
+void MidiPermissionRequest::Cancelled() {
+  context_->NotifyPermissionSet(id_, requesting_frame_, callback_, false);
+}
+
+void MidiPermissionRequest::RequestFinished() {
+  delete this;
+}
 
 ChromeMidiPermissionContext::ChromeMidiPermissionContext(Profile* profile)
     : profile_(profile),
       shutting_down_(false) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 }
 
 ChromeMidiPermissionContext::~ChromeMidiPermissionContext() {
   DCHECK(!permission_queue_controller_);
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
@@ skipping 32 matching lines @@
   // Any user's action will not be stored to content settings data base.
   if ((!requesting_frame.is_valid() && !requesting_frame.is_empty()) ||
       !embedder.is_valid()) {
     LOG(WARNING) << "Attempt to use MIDI sysex from an invalid URL: "
                  << requesting_frame << "," << embedder
                  << " (Web MIDI is not supported in popups)";
     PermissionDecided(id, requesting_frame, embedder, callback, false);
     return;
   }
 
-  DecidePermission(id, requesting_frame, embedder, callback);
+  DecidePermission(web_contents, id, requesting_frame, embedder, callback);
 }
 
 void ChromeMidiPermissionContext::CancelMidiSysExPermissionRequest(
     int render_process_id,
     int render_view_id,
     int bridge_id,
     const GURL& requesting_frame) {
   CancelPendingInfobarRequest(
       PermissionRequestID(render_process_id, render_view_id, bridge_id, 0));
 }
 
 void ChromeMidiPermissionContext::DecidePermission(
+    content::WebContents* web_contents,
     const PermissionRequestID& id,
     const GURL& requesting_frame,
     const GURL& embedder,
     const content::BrowserContext::MidiSysExPermissionCallback& callback) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
   ContentSetting content_setting =
       profile_->GetHostContentSettingsMap()->GetContentSetting(
           requesting_frame,
           embedder,
           CONTENT_SETTINGS_TYPE_MIDI_SYSEX,
           std::string());
   switch (content_setting) {
     case CONTENT_SETTING_BLOCK:
       PermissionDecided(id, requesting_frame, embedder, callback, false);
       break;
     case CONTENT_SETTING_ALLOW:
       PermissionDecided(id, requesting_frame, embedder, callback, true);
       break;
     default:
+      if (PermissionBubbleManager::Enabled()) {

[Ami GONE FROM CHROMIUM, 2014/02/06 19:51:17]

Can any code be deleted once this is always-on?
(please TODOify ifyes)

[Greg Billock, 2014/02/07 18:20:55]

Will do. I have a bug tracking this.

+        PermissionBubbleManager* bubble_manager =
+            PermissionBubbleManager::FromWebContents(web_contents);
+        bubble_manager->AddRequest(new MidiPermissionRequest(
+            this, id, requesting_frame,
+            profile_->GetPrefs()->GetString(prefs::kAcceptLanguages),
+            callback));
+        return;
+      }
+
       GetQueueController()->CreateInfoBarRequest(
           id, requesting_frame, embedder, base::Bind(
               &ChromeMidiPermissionContext::NotifyPermissionSet,
               base::Unretained(this), id, requesting_frame, callback));
   }
 }
 
 void ChromeMidiPermissionContext::PermissionDecided(
     const PermissionRequestID& id,
     const GURL& requesting_frame,
@@ skipping 31 matching lines @@
                                       CONTENT_SETTINGS_TYPE_MIDI_SYSEX));
   }
   return permission_queue_controller_.get();
 }
 
 void ChromeMidiPermissionContext::CancelPendingInfobarRequest(
     const PermissionRequestID& id) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
   if (shutting_down_)
     return;
+  // TODO(gbillock): Add support for cancellation to permission bubbles.

[Ami GONE FROM CHROMIUM, 2014/02/06 19:51:17]

What is the usability impact of this lack?

[Greg Billock, 2014/02/07 18:20:55]

Still investigating, but looks minimal, and will get fixed before the flag gets
flipped on.

   GetQueueController()->CancelInfoBarRequest(id);
 }