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

Side by Side Diff: content/browser/renderer_host/media/audio_renderer_host_unittest.cc

Issue 2561343002: Revert making trying to authorize a nonsense device id a bad_message. (Closed)
Patch Set: Created 4 years 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 unified diff | Download patch
« no previous file with comments | « content/browser/renderer_host/media/audio_output_authorization_handler_unittest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/media/audio_renderer_host.h" 5 #include "content/browser/renderer_host/media/audio_renderer_host.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <memory> 9 #include <memory>
10 10
(...skipping 591 matching lines...) Expand 10 before | Expand all | Expand 10 after
602 AssertBadMsgReported(); 602 AssertBadMsgReported();
603 } 603 }
604 604
605 TEST_F(AudioRendererHostTest, CreateDeviceWithBadSecurityOrigin) { 605 TEST_F(AudioRendererHostTest, CreateDeviceWithBadSecurityOrigin) {
606 RequestDeviceAuthorizationWithBadOrigin(kNondefaultDeviceId); 606 RequestDeviceAuthorizationWithBadOrigin(kNondefaultDeviceId);
607 Close(); 607 Close();
608 AssertBadMsgReported(); 608 AssertBadMsgReported();
609 } 609 }
610 610
611 TEST_F(AudioRendererHostTest, CreateInvalidDevice) { 611 TEST_F(AudioRendererHostTest, CreateInvalidDevice) {
612 Create(kInvalidDeviceId, url::Origin(GURL(kSecurityOrigin)), true, false); 612 Create(kInvalidDeviceId, url::Origin(GURL(kSecurityOrigin)), true, true);
613 Close(); 613 Close();
614 AssertBadMsgReported();
615 } 614 }
616 615
617 TEST_F(AudioRendererHostTest, CreateFailsForInvalidRenderFrame) { 616 TEST_F(AudioRendererHostTest, CreateFailsForInvalidRenderFrame) {
618 CreateWithInvalidRenderFrameId(); 617 CreateWithInvalidRenderFrameId();
619 Close(); 618 Close();
620 } 619 }
621 620
622 // TODO(hclam): Add tests for data conversation in low latency mode. 621 // TODO(hclam): Add tests for data conversation in low latency mode.
623 622
624 } // namespace content 623 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/renderer_host/media/audio_output_authorization_handler_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698