Factor out authorization from AudioRendererHost.

content/browser/renderer_host/media/audio_output_authorization_handler_unittest.cc

Index: content/browser/renderer_host/media/audio_output_authorization_handler_unittest.cc
diff --git a/content/browser/renderer_host/media/audio_output_authorization_handler_unittest.cc b/content/browser/renderer_host/media/audio_output_authorization_handler_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..84d883b6e1be0ee4ee63e7ad98b1c4162db9e046
--- /dev/null
+++ b/content/browser/renderer_host/media/audio_output_authorization_handler_unittest.cc
@@ -0,0 +1,206 @@
+// Copyright 2016 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.
+
+// Unit test for AudioOutputAuthorizationHandlerTest.
+
+#include "content/browser/renderer_host/media/audio_output_authorization_handler.h"
+
+#include "base/bind.h"
+#include "base/command_line.h"
+#include "base/run_loop.h"
+#include "content/public/test/mock_render_process_host.h"
+#include "content/public/test/test_browser_context.h"
+#include "content/public/test/test_browser_thread_bundle.h"
+#include "media/audio/audio_device_description.h"
+#include "media/audio/fake_audio_manager.h"
+#include "media/base/media_switches.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "url/gurl.h"
+#include "url/origin.h"
+
+using ::testing::_;
+
+const int kRenderProcessId = 1;

[o1ka, 2016/11/03 10:07:22]

hide statics in an unnamed namespace

[Max Morin, 2016/11/10 14:59:52]

Done.

+const int kRenderFrameId = 2;
+const char kSecurityOriginString[] = "http://localhost";
+const GURL kSecurityOriginGURL(kSecurityOriginString);

[o1ka, 2016/11/03 10:07:22]

only PODs can be static:
https://google.github.io/styleguide/cppguide.html#Static_and_Global_Variables

[Max Morin, 2016/11/10 14:59:52]

Done.

+const url::Origin kSecurityOrigin(kSecurityOriginGURL);
+const char kBadSecurityOriginString[] = "about:about";
+const GURL kBadSecurityOriginGURL(kBadSecurityOriginString);
+const url::Origin kBadSecurityOrigin(kBadSecurityOriginGURL);
+const char kDefaultDeviceId[] = "default";
+const char kEmptyDeviceId[] = "";
+const char kInvalidDeviceId[] = "invalid-device-id";
+const char kSalt[] = "salt";
+
+namespace content {
+
+class AudioOutputAuthorizationHandlerTest : public testing::Test {
+ public:
+  AudioOutputAuthorizationHandlerTest() {
+    audio_manager_ = media::FakeAudioManager::CreateForTesting(
+        base::ThreadTaskRunnerHandle::Get());
+    base::CommandLine::ForCurrentProcess()->AppendSwitch(
+        switches::kUseFakeDeviceForMediaStream);
+    media_stream_manager_.reset(new MediaStreamManager(audio_manager_.get()));
+    raw_nondefault_id = GetRawNondefaultFakeId();
+    hashed_nondefault_id = MediaStreamManager::GetHMACForMediaDeviceID(
+        kSalt, kSecurityOrigin, raw_nondefault_id);
+  }
+
+  ~AudioOutputAuthorizationHandlerTest() override {}
+
+ protected:
+  void SetUp() override {}
+
+  void TearDown() override {}
+
+  std::unique_ptr<MediaStreamManager> media_stream_manager_;
+  TestBrowserThreadBundle thread_bundle_;

[o1ka, 2016/11/03 10:07:22]

All the browser threads are emulated with a single thread here, so technically
you run it all on UI thread.
https://cs.chromium.org/chromium/src/content/public/test/test_browser_thread_...

Probably we need to set up multi-threaded testing? WDYT?

[Max Morin, 2016/11/10 14:59:52]

Done.

+  media::ScopedAudioManagerPtr audio_manager_;
+  std::string raw_nondefault_id;
+  std::string hashed_nondefault_id;
+
+ private:
+  std::string GetRawNondefaultFakeId() {
+    MediaDeviceInfoArray devices;
+    MediaDevicesManager::BoolDeviceTypes devices_to_enumerate;
+    devices_to_enumerate[MEDIA_DEVICE_TYPE_AUDIO_OUTPUT] = true;
+
+    media_stream_manager_->media_devices_manager()->EnumerateDevices(
+        devices_to_enumerate,
+        base::Bind(
+            [](MediaDeviceInfoArray* out,
+               const MediaDeviceEnumeration& result) {
+              *out = result[MediaDeviceType::MEDIA_DEVICE_TYPE_AUDIO_OUTPUT];
+            },
+            &devices));
+    base::RunLoop().RunUntilIdle();
+    return devices[1].device_id;  // devices[0] is default.
+  }
+
+  DISALLOW_COPY_AND_ASSIGN(AudioOutputAuthorizationHandlerTest);
+};
+
+struct MockListener {
+  MOCK_METHOD3(MockAuthorizationCallback,
+               void(media::OutputDeviceStatus status,
+                    const media::AudioParameters& params,
+                    const std::string& id));
+  AudioOutputAuthorizationHandler::AuthorizationCompletedCallback
+  GetCallback() {
+    return base::Bind(&MockListener::MockAuthorizationCallback,
+                      base::Unretained(this));
+  }
+};
+
+TEST_F(AudioOutputAuthorizationHandlerTest, AuthorizeDefaultDevice_Ok) {
+  AudioOutputAuthorizationHandler handler(media_stream_manager_.get(),
+                                          kRenderProcessId, kSalt);
+
+  MockListener listener;
+  EXPECT_CALL(listener,
+              MockAuthorizationCallback(media::OUTPUT_DEVICE_STATUS_OK, _,
+                                        kDefaultDeviceId))
+      .Times(1);
+
+  handler.RequestDeviceAuthorization(kRenderFrameId, /*session_id*/ 0,
+                                     kDefaultDeviceId, kSecurityOrigin,
+                                     listener.GetCallback());
+  base::RunLoop().RunUntilIdle();
+}
+
+TEST_F(AudioOutputAuthorizationHandlerTest,
+       AuthorizeDefaultDeviceByEmptyId_Ok) {
+  AudioOutputAuthorizationHandler handler(media_stream_manager_.get(),
+                                          kRenderProcessId, kSalt);
+
+  MockListener listener;
+  EXPECT_CALL(listener,
+              MockAuthorizationCallback(media::OUTPUT_DEVICE_STATUS_OK, _,
+                                        kDefaultDeviceId))
+      .Times(1);
+
+  handler.RequestDeviceAuthorization(kRenderFrameId, /*session_id*/ 0,
+                                     kEmptyDeviceId, kSecurityOrigin,
+                                     listener.GetCallback());
+  base::RunLoop().RunUntilIdle();
+}
+
+TEST_F(AudioOutputAuthorizationHandlerTest,
+       AuthorizeNonDefaultDeviceIdWithoutPermission_NotAuthorized) {
+  AudioOutputAuthorizationHandler handler(media_stream_manager_.get(),
+                                          kRenderProcessId, kSalt);
+  std::string id = MediaStreamManager::GetHMACForMediaDeviceID(
+      kSalt, kSecurityOrigin, raw_nondefault_id);
+
+  MockListener listener;
+  EXPECT_CALL(listener,
+              MockAuthorizationCallback(
+                  media::OUTPUT_DEVICE_STATUS_ERROR_NOT_AUTHORIZED, _, _))
+      .Times(1);
+
+  handler.RequestDeviceAuthorization(kRenderFrameId, /*session_id*/ 0, id,
+                                     kSecurityOrigin, listener.GetCallback());
+  base::RunLoop().RunUntilIdle();
+}
+
+TEST_F(AudioOutputAuthorizationHandlerTest,
+       AuthorizeNonDefaultDeviceIdWithPermission_Ok) {
+  AudioOutputAuthorizationHandler handler(media_stream_manager_.get(),
+                                          kRenderProcessId, kSalt);
+  // Override permissions check.
+  handler.GetMediaDevicesPermissionCheckerForTesting()
+      .OverridePermissionsForTesting(true);
+
+  MockListener listener;
+  EXPECT_CALL(listener,
+              MockAuthorizationCallback(media::OUTPUT_DEVICE_STATUS_OK, _,
+                                        raw_nondefault_id))
+      .Times(1);
+
+  handler.RequestDeviceAuthorization(kRenderFrameId, /*session_id*/ 0,
+                                     hashed_nondefault_id, kSecurityOrigin,
+                                     listener.GetCallback());
+  base::RunLoop().RunUntilIdle();
+}
+
+TEST_F(AudioOutputAuthorizationHandlerTest,
+       AuthorizeInvalidDeviceId_BadMessage) {
+  TestBrowserContext context;
+  MockRenderProcessHost RPH(&context);
+  AudioOutputAuthorizationHandler handler(media_stream_manager_.get(),
+                                          RPH.GetID(), kSalt);
+
+  MockListener listener;
+  EXPECT_CALL(listener, MockAuthorizationCallback(_, _, _)).Times(0);
+
+  handler.RequestDeviceAuthorization(kRenderFrameId, /*session_id*/ 0,
+                                     kInvalidDeviceId, kSecurityOrigin,
+                                     listener.GetCallback());
+  base::RunLoop().RunUntilIdle();
+
+  EXPECT_EQ(RPH.bad_msg_count(), 1);

[o1ka, 2016/11/03 10:07:22]

nit: expect it to be 0 at the beginning of the test

[Max Morin, 2016/11/10 14:59:52]

Done.

+}
+
+TEST_F(AudioOutputAuthorizationHandlerTest,
+       AuthorizeNondefaultDeviceIdWithBadOrigin_BadMessage) {
+  TestBrowserContext context;
+  MockRenderProcessHost RPH(&context);
+  AudioOutputAuthorizationHandler handler(media_stream_manager_.get(),
+                                          RPH.GetID(), kSalt);
+
+  MockListener listener;
+  EXPECT_CALL(listener, MockAuthorizationCallback(_, _, _)).Times(0);
+
+  handler.RequestDeviceAuthorization(kRenderFrameId, /*session_id*/ 0,
+                                     hashed_nondefault_id, kBadSecurityOrigin,
+                                     listener.GetCallback());
+  base::RunLoop().RunUntilIdle();
+
+  EXPECT_EQ(RPH.bad_msg_count(), 1);
+}
+
+}  // namespace content