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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutMedia.cpp

Issue 2510353004: Deprecating AutoplayExperimentHelper (Closed)
Patch Set: rebased 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/core/layout/LayoutMedia.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutMedia.cpp b/third_party/WebKit/Source/core/layout/LayoutMedia.cpp
index e85413688c4df85a8552f5c983d458e03b198e69..7cba355293374fce0d2d1efc0eec62fb296e379a 100644
--- a/third_party/WebKit/Source/core/layout/LayoutMedia.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutMedia.cpp
@@ -130,34 +130,6 @@ bool LayoutMedia::isChildAllowed(LayoutObject* child,
void LayoutMedia::paintReplaced(const PaintInfo&, const LayoutPoint&) const {}
-void LayoutMedia::willBeDestroyed() {
- if (view())
- view()->unregisterMediaForPositionChangeNotification(*this);
- LayoutImage::willBeDestroyed();
-}
-
-void LayoutMedia::insertedIntoTree() {
- LayoutImage::insertedIntoTree();
-
- // Note that if we don't want them and aren't registered, then this
- // will do nothing.
- if (HTMLMediaElement* element = mediaElement())
- element->updatePositionNotificationRegistration();
-}
-
-void LayoutMedia::notifyPositionMayHaveChanged(const IntRect& visibleRect) {
- // Tell our element about it.
- if (HTMLMediaElement* element = mediaElement())
- element->notifyPositionMayHaveChanged(visibleRect);
-}
-
-void LayoutMedia::setRequestPositionUpdates(bool want) {
- if (want)
- view()->registerMediaForPositionChangeNotification(*this);
- else
- view()->unregisterMediaForPositionChangeNotification(*this);
-}
-
LayoutUnit LayoutMedia::computePanelWidth(const LayoutRect& mediaRect) const {
// TODO(mlamouri): we don't know if the main frame has an horizontal scrollbar
// if it is out of process. See https://crbug.com/662480
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutMedia.h ('k') | third_party/WebKit/Source/core/layout/LayoutView.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698