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

Unified Diff: content/browser/renderer_host/browser_compositor_view_mac.h

Issue 2776973002: DelegatedFrameHost should not create an ExternalBeginFrameSource (Closed)
Patch Set: c Created 3 years, 9 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 | « no previous file | content/browser/renderer_host/browser_compositor_view_mac.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/browser_compositor_view_mac.h
diff --git a/content/browser/renderer_host/browser_compositor_view_mac.h b/content/browser/renderer_host/browser_compositor_view_mac.h
index d62da98df7552b1d7048eb1125b61ab8dfb805ec..aa4d989a09b86a481869f8c6b79fae07a26524b7 100644
--- a/content/browser/renderer_host/browser_compositor_view_mac.h
+++ b/content/browser/renderer_host/browser_compositor_view_mac.h
@@ -43,8 +43,7 @@ class BrowserCompositorMacClient {
// is visible.
// - The RenderWidgetHostViewMac that is used to display these frames is
// attached to the NSView hierarchy of an NSWindow.
-class BrowserCompositorMac : public cc::BeginFrameObserver,
- public DelegatedFrameHostClient {
+class BrowserCompositorMac : public DelegatedFrameHostClient {
public:
BrowserCompositorMac(
ui::AcceleratedWidgetMacNSView* accelerated_widget_mac_ns_view,
@@ -112,13 +111,8 @@ class BrowserCompositorMac : public cc::BeginFrameObserver,
int compositor_frame_sink_id,
bool is_swap_ack,
const cc::ReturnedResourceArray& resources) override;
- void SetBeginFrameSource(cc::BeginFrameSource* source) override;
- bool IsAutoResizeEnabled() const override;
-
- // cc::BeginFrameObserver implementation.
void OnBeginFrame(const cc::BeginFrameArgs& args) override;
- const cc::BeginFrameArgs& LastUsedBeginFrameArgs() const override;
- void OnBeginFrameSourcePausedChanged(bool paused) override;
+ bool IsAutoResizeEnabled() const override;
private:
// The state of |delegated_frame_host_| and |recyclable_compositor_| to
@@ -179,11 +173,6 @@ class BrowserCompositorMac : public cc::BeginFrameObserver,
bool has_transparent_background_ = false;
- // The begin frame source being observed. Null if none.
- cc::BeginFrameSource* begin_frame_source_ = nullptr;
- cc::BeginFrameArgs last_begin_frame_args_;
- bool needs_begin_frames_ = false;
-
base::WeakPtrFactory<BrowserCompositorMac> weak_factory_;
};
« no previous file with comments | « no previous file | content/browser/renderer_host/browser_compositor_view_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698