Send enter / exit fullscreen signal to AVDA

media/base/overlay_info.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 "media/base/overlay_info.h"
+
+namespace media {
+
+OverlayInfo::OverlayInfo() = default;
+OverlayInfo::OverlayInfo(const OverlayInfo&) = default;
+
+// static
+OverlayInfo OverlayInfo::SurfaceChange(int surface_id) {
+  OverlayInfo info;
+  info.surface_id = surface_id;
+  return info;
+}
+
+// static
+OverlayInfo OverlayInfo::RoutingTokenChange(
+    base::Optional<base::UnguessableToken> token) {
+  OverlayInfo info;
+  info.routing_token = token;
+  return info;
+}
+
+OverlayInfo& OverlayInfo::operator|=(const OverlayInfo& other) {
+  if (other.surface_id)
+    surface_id = other.surface_id;
+  if (other.routing_token)

[tguilbert, 2017/05/24 19:14:54]

Is there a reason why |is_fullscreen| is not copied?

[liberato (no reviews please), 2017/05/24 21:21:07]

because it's error-prone.  :)  good catch.

i really don't like this pattern.  too bad that one can't "operator|=() =
default" with base::Optional supporting |= .

+    routing_token = other.routing_token;
+
+  return *this;
+}
+
+}  // namespace media