Allow web-midi permission to be granted to insecure origins

chrome/browser/media/midi_sysex_permission_context_unittest.cc

 // Copyright 2017 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/midi_sysex_permission_context.h"
 #include "base/bind.h"
 #include "base/macros.h"
 #include "build/build_config.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
-#include "chrome/browser/media/midi_permission_context.h"
 #include "chrome/browser/permissions/permission_request_id.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "chrome/test/base/testing_profile.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
 #include "components/content_settings/core/common/content_settings.h"
 #include "components/content_settings/core/common/content_settings_types.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/test/mock_render_process_host.h"
 #include "content/public/test/web_contents_tester.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 #if defined(OS_ANDROID)
 #include "chrome/browser/infobars/infobar_service.h"
 #else
 #include "chrome/browser/permissions/permission_request_manager.h"
 #endif
 
 namespace {
 
-class TestPermissionContext : public MidiPermissionContext {
+class TestPermissionContext : public MidiSysexPermissionContext {
  public:
   explicit TestPermissionContext(Profile* profile)
-      : MidiPermissionContext(profile),
+      : MidiSysexPermissionContext(profile),
         permission_set_(false),
         permission_granted_(false),
         tab_context_updated_(false) {}
 
   ~TestPermissionContext() override {}
 
   bool permission_granted() { return permission_granted_; }
 
   bool permission_set() { return permission_set_; }
 
@@ skipping 12 matching lines @@
   }
 
  private:
   bool permission_set_;
   bool permission_granted_;
   bool tab_context_updated_;
 };
 
 }  // anonymous namespace
 
-class MidiPermissionContextTests : public ChromeRenderViewHostTestHarness {
+class MidiSysexPermissionContextTests : public ChromeRenderViewHostTestHarness {
  protected:
-  MidiPermissionContextTests() = default;
+  MidiSysexPermissionContextTests() = default;
 
  private:
   // ChromeRenderViewHostTestHarness:
   void SetUp() override {
     ChromeRenderViewHostTestHarness::SetUp();
 #if defined(OS_ANDROID)
     InfoBarService::CreateForWebContents(web_contents());
 #else
     PermissionRequestManager::CreateForWebContents(web_contents());
 #endif
   }
 
-  DISALLOW_COPY_AND_ASSIGN(MidiPermissionContextTests);
+  DISALLOW_COPY_AND_ASSIGN(MidiSysexPermissionContextTests);
 };
 
-// Web MIDI permission should be denied for insecure origin.
-TEST_F(MidiPermissionContextTests, TestInsecureRequestingUrl) {
+// Web MIDI sysex permission should be denied for insecure origin.
+TEST_F(MidiSysexPermissionContextTests, TestInsecureRequestingUrl) {
   TestPermissionContext permission_context(profile());
   GURL url("http://www.example.com");
   content::WebContentsTester::For(web_contents())->NavigateAndCommit(url);
 
   const PermissionRequestID id(web_contents()->GetRenderProcessHost()->GetID(),
                                web_contents()->GetMainFrame()->GetRoutingID(),
                                -1);
   permission_context.RequestPermission(
       web_contents(), id, url, true,
       base::Bind(&TestPermissionContext::TrackPermissionDecision,
                  base::Unretained(&permission_context)));
 
   EXPECT_TRUE(permission_context.permission_set());
   EXPECT_FALSE(permission_context.permission_granted());
   EXPECT_TRUE(permission_context.tab_context_updated());
 
   ContentSetting setting =
       HostContentSettingsMapFactory::GetForProfile(profile())
           ->GetContentSetting(url.GetOrigin(), url.GetOrigin(),
                               CONTENT_SETTINGS_TYPE_MIDI_SYSEX, std::string());
   EXPECT_EQ(CONTENT_SETTING_ASK, setting);
 }
 
-// Web MIDI permission status should be denied for insecure origin.
-TEST_F(MidiPermissionContextTests, TestInsecureQueryingUrl) {
+// Web MIDI sysex permission status should be denied for insecure origin.
+TEST_F(MidiSysexPermissionContextTests, TestInsecureQueryingUrl) {
   TestPermissionContext permission_context(profile());
   GURL insecure_url("http://www.example.com");
   GURL secure_url("https://www.example.com");
 
   // Check that there is no saved content settings.
   EXPECT_EQ(CONTENT_SETTING_ASK,
             HostContentSettingsMapFactory::GetForProfile(profile())
                 ->GetContentSetting(
                     insecure_url.GetOrigin(), insecure_url.GetOrigin(),
                     CONTENT_SETTINGS_TYPE_MIDI_SYSEX, std::string()));
@@ skipping 13 matching lines @@
                 .GetPermissionStatus(nullptr /* render_frame_host */,
                                      insecure_url, insecure_url)
                 .content_setting);
 
   EXPECT_EQ(CONTENT_SETTING_BLOCK,
             permission_context
                 .GetPermissionStatus(nullptr /* render_frame_host */,
                                      insecure_url, secure_url)
                 .content_setting);
 }