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

Side by Side Diff: media/base/overlay_info_unittest.cc

Issue 2892083002: Send enter / exit fullscreen signal to AVDA (Closed)
Patch Set: more cl feedback Created 3 years, 7 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "media/base/overlay_info.h"
6 #include "testing/gtest/include/gtest/gtest.h"
7
8 namespace media {
9
10 using UnguessableToken = base::UnguessableToken;
11 using RoutingToken = OverlayInfo::RoutingToken;
12
13 // Test that MergeWith handles |member_ptr|. |some_value| and |other_value|
14 // should be two unequal values for |MemberType|.
15 template <typename MemberType>
16 void MergeWithHelper(base::Optional<MemberType> OverlayInfo::*member_ptr,
17 MemberType some_value,
18 MemberType other_value) {
19 // Sanity.
20 EXPECT_NE(some_value, other_value);
21
22 OverlayInfo source;
23 source.*member_ptr = some_value;
24
25 OverlayInfo dest;
26
27 // Make sure that it works with no value.
28 dest.MergeWith(source);
29 EXPECT_EQ(source.*member_ptr, dest.*member_ptr);
30
31 // Try replacing an existing value.
32 dest.*member_ptr = other_value;
33 dest.MergeWith(source);
34 EXPECT_EQ(source.*member_ptr, dest.*member_ptr);
35 }
36
37 TEST(OverlayInfo, MergeWithHandlesSurfaceId) {
38 MergeWithHelper(&OverlayInfo::surface_id, 123, 234);
39 }
40
41 TEST(OverlayInfo, MergeWithHandlesRoutingToken) {
42 MergeWithHelper(&OverlayInfo::routing_token,
43 RoutingToken(UnguessableToken::Create()),
44 RoutingToken(UnguessableToken::Create()));
45 }
46
47 TEST(OverlayInfo, MergeWithHandlesIsFullscreen) {
48 MergeWithHelper(&OverlayInfo::is_fullscreen, true, false);
49 }
50
51 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698