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

Unified Diff: content/public/browser/render_widget_host_view.h

Issue 853883007: Add occlusion support to WebContentsImpl and RenderWidgetHostView (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clean up Created 5 years, 11 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/public/browser/render_widget_host_view.h
diff --git a/content/public/browser/render_widget_host_view.h b/content/public/browser/render_widget_host_view.h
index 6d97fd898ef8fba1ae5dc1034ab169d6deddc8ee..22ec856b9fd99cb2d98f30e19b031b1752e21080 100644
--- a/content/public/browser/render_widget_host_view.h
+++ b/content/public/browser/render_widget_host_view.h
@@ -93,6 +93,12 @@ class CONTENT_EXPORT RenderWidgetHostView {
// Whether the view is showing.
virtual bool IsShowing() = 0;
+ // Indicates if the view is currently occluded and the view may be suspended.
Avi (use Gerrit) 2015/01/16 20:15:13 You might want to give an "e.g." for occlusion her
ccameron 2015/01/16 21:43:40 Done.
+ // If Show() is called on a view then its state is re-set to being
+ // un-occluded. These calls do not necessarily made in pairs.
Avi (use Gerrit) 2015/01/16 20:15:13 "do not necessarily made" is weird grammar; clean
ccameron 2015/01/16 21:43:40 Fixed. Also expanded on the points of being re-set
+ virtual void WasUnOccluded() = 0;
+ virtual void WasOccluded() = 0;
+
// Retrieve the bounds of the View, in screen coordinates.
virtual gfx::Rect GetViewBounds() const = 0;

Powered by Google App Engine
This is Rietveld 408576698