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

Unified Diff: chrome/browser/media/midi_permission_context_unittest.cc

Issue 2686463002: Add a source to the result of PermissionContextBase::GetPermissionStatus (Closed)
Patch Set: Created 3 years, 10 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_context_unittest.cc
diff --git a/chrome/browser/media/midi_permission_context_unittest.cc b/chrome/browser/media/midi_permission_context_unittest.cc
index 6a73e470bab58ea80f671dc97bf98bc6f10fee4e..49f653bd2ac96544c271eb46d440382bbecf5ec5 100644
--- a/chrome/browser/media/midi_permission_context_unittest.cc
+++ b/chrome/browser/media/midi_permission_context_unittest.cc
@@ -142,8 +142,12 @@ TEST_F(MidiPermissionContextTests, TestInsecureQueryingUrl) {
CONTENT_SETTINGS_TYPE_MIDI_SYSEX,
std::string()));
- EXPECT_EQ(CONTENT_SETTING_BLOCK, permission_context.GetPermissionStatus(
- insecure_url, insecure_url));
- EXPECT_EQ(CONTENT_SETTING_BLOCK, permission_context.GetPermissionStatus(
- insecure_url, secure_url));
+ PermissionResult result =
+ permission_context.GetPermissionStatus(insecure_url, insecure_url);
+ EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+ EXPECT_EQ(PermissionStatusSource::INSECURE_ORIGIN, result.source);
+
+ result = permission_context.GetPermissionStatus(insecure_url, secure_url);
+ EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+ EXPECT_EQ(PermissionStatusSource::INSECURE_ORIGIN, result.source);
}

Powered by Google App Engine
This is Rietveld 408576698