FrameSinkVideoCapture clean-up: Remove unused APIs, style fixes.
1. Remove the unused viz.mojom.FrameSinkVideoConsumer.OnTargetLost()
API.
2. Fix the viz.mojom.FrameSinkVideoCapturer.ChangeTarget() to accept an
optional FrameSinkId (original intended behavior of the API before new
struct traits restrictions on "null" FrameSinkIds).
3. C++ Style: Fixed argument ordering in a few methods of viz::FSVCImpl
so that the output argument would come last.
4. Ran lint on all files changed, and made fixes (mostly
include-what-you-use).
Bug: 754872
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: Idbceb49089811dd6f8494f8ee17d578239108866
Reviewed-on: https://chromium-review.googlesource.com/1090220
Commit-Queue: Yuri Wiitala <[email protected]>
Reviewed-by: Daniel Cheng <[email protected]>
Reviewed-by: Andrey Kosyakov <[email protected]>
Reviewed-by: Xiangjun Zhang <[email protected]>
Cr-Commit-Position: refs/heads/master@{#568748}diff --git a/chrome/browser/devtools/devtools_eye_dropper.h b/chrome/browser/devtools/devtools_eye_dropper.h
index c120c98..0dde971e 100644
--- a/chrome/browser/devtools/devtools_eye_dropper.h
+++ b/chrome/browser/devtools/devtools_eye_dropper.h
@@ -5,6 +5,8 @@
#ifndef CHROME_BROWSER_DEVTOOLS_DEVTOOLS_EYE_DROPPER_H_
#define CHROME_BROWSER_DEVTOOLS_DEVTOOLS_EYE_DROPPER_H_
+#include <memory>
+
#include "base/callback.h"
#include "base/macros.h"
#include "components/viz/host/client_frame_sink_video_capturer.h"
@@ -52,7 +54,6 @@
const gfx::Rect& update_rect,
const gfx::Rect& content_rect,
viz::mojom::FrameSinkVideoConsumerFrameCallbacksPtr callbacks) override;
- void OnTargetLost(const viz::FrameSinkId& frame_sink_id) override;
void OnStopped() override;
// This object keeps the shared memory that backs |frame_| mapped.
