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

Unified Diff: third_party/WebKit/Source/web/WebViewImpl.h

Issue 2550703002: Move pending state from FullscreenController to Fullscreen (Closed)
Patch Set: tests and documentation Created 4 years 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: third_party/WebKit/Source/web/WebViewImpl.h
diff --git a/third_party/WebKit/Source/web/WebViewImpl.h b/third_party/WebKit/Source/web/WebViewImpl.h
index c03f41a3cb2d46812e6a32061af911b6b3b53031..66a0cfa7bd7eaba8db52d73f268d62a72507fae9 100644
--- a/third_party/WebKit/Source/web/WebViewImpl.h
+++ b/third_party/WebKit/Source/web/WebViewImpl.h
@@ -445,8 +445,8 @@ class WEB_EXPORT WebViewImpl final
return m_fakePageScaleAnimationUseAnchor;
}
- void enterFullscreenForElement(Element*);
- void exitFullscreen(LocalFrame*);
+ void enterFullscreen(LocalFrame&);
+ void exitFullscreen(LocalFrame&);
void fullscreenElementChanged(Element*, Element*);
// Exposed for the purpose of overriding device metrics.
@@ -697,7 +697,7 @@ class WEB_EXPORT WebViewImpl final
WebGestureDevice m_flingSourceDevice;
Vector<std::unique_ptr<LinkHighlightImpl>> m_linkHighlights;
std::unique_ptr<CompositorAnimationTimeline> m_linkHighlightsTimeline;
- Persistent<FullscreenController> m_fullscreenController;
+ std::unique_ptr<FullscreenController> m_fullscreenController;
WebColor m_baseBackgroundColor;
WebColor m_backgroundColorOverride;

Powered by Google App Engine
This is Rietveld 408576698