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

Unified Diff: cc/test/fake_compositor_frame_sink.h

Issue 2349743004: cc: Remove things from OutputSurface and CompositorFrameSink. (Closed)
Patch Set: delete-stuff-cfs: comment-and-rebase Created 4 years, 3 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
« no previous file with comments | « cc/surfaces/display.cc ('k') | cc/test/fake_compositor_frame_sink.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/test/fake_compositor_frame_sink.h
diff --git a/cc/test/fake_compositor_frame_sink.h b/cc/test/fake_compositor_frame_sink.h
index 5c09bf18b73b4dfa17ca574d572c0d71317ff601..598a8e4f152ec41d3f82d1caf92d76924e9d469d 100644
--- a/cc/test/fake_compositor_frame_sink.h
+++ b/cc/test/fake_compositor_frame_sink.h
@@ -26,31 +26,25 @@ class FakeCompositorFrameSink : public CompositorFrameSink {
~FakeCompositorFrameSink() override;
static std::unique_ptr<FakeCompositorFrameSink> Create3d() {
- return base::WrapUnique(
- new FakeCompositorFrameSink(TestContextProvider::Create(),
- TestContextProvider::CreateWorker(), true));
+ return base::WrapUnique(new FakeCompositorFrameSink(
+ TestContextProvider::Create(), TestContextProvider::CreateWorker()));
}
static std::unique_ptr<FakeCompositorFrameSink> Create3d(
scoped_refptr<TestContextProvider> context_provider) {
return base::WrapUnique(new FakeCompositorFrameSink(
- context_provider, TestContextProvider::CreateWorker(), true));
+ context_provider, TestContextProvider::CreateWorker()));
}
static std::unique_ptr<FakeCompositorFrameSink> Create3d(
std::unique_ptr<TestWebGraphicsContext3D> context) {
return base::WrapUnique(new FakeCompositorFrameSink(
TestContextProvider::Create(std::move(context)),
- TestContextProvider::CreateWorker(), true));
+ TestContextProvider::CreateWorker()));
}
static std::unique_ptr<FakeCompositorFrameSink> CreateSoftware() {
- return base::WrapUnique(
- new FakeCompositorFrameSink(nullptr, nullptr, nullptr, true));
- }
-
- void set_max_frames_pending(int max) {
- capabilities_.max_frames_pending = max;
+ return base::WrapUnique(new FakeCompositorFrameSink(nullptr, nullptr));
}
CompositorFrame* last_sent_frame() { return last_sent_frame_.get(); }
@@ -62,34 +56,10 @@ class FakeCompositorFrameSink : public CompositorFrameSink {
bool BindToClient(CompositorFrameSinkClient* client) override;
void DetachFromClient() override;
- void set_framebuffer(GLint framebuffer, GLenum format) {
- framebuffer_ = framebuffer;
- framebuffer_format_ = format;
- }
- void BindFramebuffer() override;
- uint32_t GetFramebufferCopyTextureFormat() override;
-
const TransferableResourceArray& resources_held_by_parent() {
return resources_held_by_parent_;
}
- bool HasExternalStencilTest() const override;
-
- bool SurfaceIsSuspendForRecycle() const override;
-
- OverlayCandidateValidator* GetOverlayCandidateValidator() const override;
- void SetOverlayCandidateValidator(OverlayCandidateValidator* validator) {
- overlay_candidate_validator_ = validator;
- }
-
- void set_has_external_stencil_test(bool has_test) {
- has_external_stencil_test_ = has_test;
- }
-
- void set_suspended_for_recycle(bool suspended) {
- suspended_for_recycle_ = suspended;
- }
-
gfx::Rect last_swap_rect() const {
DCHECK(last_swap_rect_valid_);
return last_swap_rect_;
@@ -100,24 +70,12 @@ class FakeCompositorFrameSink : public CompositorFrameSink {
protected:
FakeCompositorFrameSink(
scoped_refptr<ContextProvider> context_provider,
- scoped_refptr<ContextProvider> worker_context_provider,
- bool delegated_rendering);
-
- FakeCompositorFrameSink(
- scoped_refptr<ContextProvider> context_provider,
- scoped_refptr<ContextProvider> worker_context_provider,
- std::unique_ptr<SoftwareOutputDevice> software_device,
- bool delegated_rendering);
+ scoped_refptr<ContextProvider> worker_context_provider);
CompositorFrameSinkClient* client_ = nullptr;
std::unique_ptr<CompositorFrame> last_sent_frame_;
size_t num_sent_frames_ = 0;
- bool has_external_stencil_test_ = false;
- bool suspended_for_recycle_ = false;
- GLint framebuffer_ = 0;
- GLenum framebuffer_format_ = 0;
TransferableResourceArray resources_held_by_parent_;
- OverlayCandidateValidator* overlay_candidate_validator_ = nullptr;
bool last_swap_rect_valid_ = false;
gfx::Rect last_swap_rect_;
};
« no previous file with comments | « cc/surfaces/display.cc ('k') | cc/test/fake_compositor_frame_sink.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698