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

Unified Diff: content/browser/compositor/browser_compositor_overlay_candidate_validator_android.cc

Issue 1931213002: display_compositor: Move CompositorOverlayCandidateValidator from content. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@DONE_2016.04.28_dc-overlay-candidate-validator
Patch Set: . Created 4 years, 8 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: content/browser/compositor/browser_compositor_overlay_candidate_validator_android.cc
diff --git a/content/browser/compositor/browser_compositor_overlay_candidate_validator_android.cc b/content/browser/compositor/browser_compositor_overlay_candidate_validator_android.cc
deleted file mode 100644
index e04a241550db95e20a08e73bbca8c7a0d9aec3ee..0000000000000000000000000000000000000000
--- a/content/browser/compositor/browser_compositor_overlay_candidate_validator_android.cc
+++ /dev/null
@@ -1,54 +0,0 @@
-// Copyright 2015 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 "content/browser/compositor/browser_compositor_overlay_candidate_validator_android.h"
-
-#include <memory>
-
-#include "base/memory/ptr_util.h"
-#include "cc/output/overlay_processor.h"
-#include "cc/output/overlay_strategy_underlay.h"
-#include "ui/gfx/geometry/rect_conversions.h"
-
-namespace content {
-
-BrowserCompositorOverlayCandidateValidatorAndroid::
- BrowserCompositorOverlayCandidateValidatorAndroid() {}
-
-BrowserCompositorOverlayCandidateValidatorAndroid::
- ~BrowserCompositorOverlayCandidateValidatorAndroid() {}
-
-void BrowserCompositorOverlayCandidateValidatorAndroid::GetStrategies(
- cc::OverlayProcessor::StrategyList* strategies) {
- strategies->push_back(
- base::WrapUnique(new cc::OverlayStrategyUnderlay(this)));
-}
-
-void BrowserCompositorOverlayCandidateValidatorAndroid::CheckOverlaySupport(
- cc::OverlayCandidateList* candidates) {
- // There should only be at most a single overlay candidate: the video quad.
- // There's no check that the presented candidate is really a video frame for
- // a fullscreen video. Instead it's assumed that if a quad is marked as
- // overlayable, it's a fullscreen video quad.
- DCHECK_LE(candidates->size(), 1u);
-
- if (!candidates->empty()) {
- cc::OverlayCandidate& candidate = candidates->front();
- candidate.display_rect =
- gfx::RectF(gfx::ToEnclosingRect(candidate.display_rect));
- candidate.overlay_handled = true;
- candidate.plane_z_order = -1;
- }
-}
-
-bool BrowserCompositorOverlayCandidateValidatorAndroid::AllowCALayerOverlays() {
- return false;
-}
-
-// Overlays will still be allowed when software mirroring is enabled, even
-// though they won't appear in the mirror.
-void BrowserCompositorOverlayCandidateValidatorAndroid::SetSoftwareMirrorMode(
- bool enabled) {}
-
-} // namespace content

Powered by Google App Engine
This is Rietveld 408576698