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

Side by Side Diff: content/browser/frame_host/render_widget_host_view_child_frame.h

Issue 2702153003: [content] Fix background color update handling in RWHVAura. (Closed)
Patch Set: Remove GetBackgroundOpaque and base impls of accessors. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_WIDGET_HOST_VIEW_CHILD_FRAME_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_WIDGET_HOST_VIEW_CHILD_FRAME_H_
6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_WIDGET_HOST_VIEW_CHILD_FRAME_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_WIDGET_HOST_VIEW_CHILD_FRAME_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 bool IsSurfaceAvailableForCopy() const override; 80 bool IsSurfaceAvailableForCopy() const override;
81 void Show() override; 81 void Show() override;
82 void Hide() override; 82 void Hide() override;
83 bool IsShowing() override; 83 bool IsShowing() override;
84 gfx::Rect GetViewBounds() const override; 84 gfx::Rect GetViewBounds() const override;
85 gfx::Size GetVisibleViewportSize() const override; 85 gfx::Size GetVisibleViewportSize() const override;
86 gfx::Vector2dF GetLastScrollOffset() const override; 86 gfx::Vector2dF GetLastScrollOffset() const override;
87 gfx::NativeView GetNativeView() const override; 87 gfx::NativeView GetNativeView() const override;
88 gfx::NativeViewAccessible GetNativeViewAccessible() override; 88 gfx::NativeViewAccessible GetNativeViewAccessible() override;
89 void SetBackgroundColor(SkColor color) override; 89 void SetBackgroundColor(SkColor color) override;
90 SkColor background_color() const override;
90 gfx::Size GetPhysicalBackingSize() const override; 91 gfx::Size GetPhysicalBackingSize() const override;
91 bool IsMouseLocked() override; 92 bool IsMouseLocked() override;
92 void SetNeedsBeginFrames(bool needs_begin_frames) override; 93 void SetNeedsBeginFrames(bool needs_begin_frames) override;
93 94
94 // RenderWidgetHostViewBase implementation. 95 // RenderWidgetHostViewBase implementation.
95 void InitAsPopup(RenderWidgetHostView* parent_host_view, 96 void InitAsPopup(RenderWidgetHostView* parent_host_view,
96 const gfx::Rect& bounds) override; 97 const gfx::Rect& bounds) override;
97 void InitAsFullscreen(RenderWidgetHostView* reference_host_view) override; 98 void InitAsFullscreen(RenderWidgetHostView* reference_host_view) override;
98 void UpdateCursor(const WebCursor& cursor) override; 99 void UpdateCursor(const WebCursor& cursor) override;
99 void SetIsLoading(bool is_loading) override; 100 void SetIsLoading(bool is_loading) override;
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
259 // Since frame-drawn callbacks are "fire once", we use std::deque to make 260 // Since frame-drawn callbacks are "fire once", we use std::deque to make
260 // it convenient to swap() when processing the list. 261 // it convenient to swap() when processing the list.
261 FrameSwappedCallbackList frame_swapped_callbacks_; 262 FrameSwappedCallbackList frame_swapped_callbacks_;
262 263
263 // The begin frame source being observed. Null if none. 264 // The begin frame source being observed. Null if none.
264 cc::BeginFrameSource* begin_frame_source_; 265 cc::BeginFrameSource* begin_frame_source_;
265 cc::BeginFrameArgs last_begin_frame_args_; 266 cc::BeginFrameArgs last_begin_frame_args_;
266 // The surface client ID of the parent RenderWidgetHostView. 0 if none. 267 // The surface client ID of the parent RenderWidgetHostView. 0 if none.
267 cc::FrameSinkId parent_frame_sink_id_; 268 cc::FrameSinkId parent_frame_sink_id_;
268 269
270 // The background color of the widget.
271 SkColor background_color_;
272
269 base::WeakPtrFactory<RenderWidgetHostViewChildFrame> weak_factory_; 273 base::WeakPtrFactory<RenderWidgetHostViewChildFrame> weak_factory_;
270 DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewChildFrame); 274 DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewChildFrame);
271 }; 275 };
272 276
273 } // namespace content 277 } // namespace content
274 278
275 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_WIDGET_HOST_VIEW_CHILD_FRAME_H_ 279 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_WIDGET_HOST_VIEW_CHILD_FRAME_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698