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

Unified Diff: third_party/WebKit/Source/core/frame/FrameView.h

Issue 1513573013: Don't call LayoutPart::widgetPositionsUpdated unnecessarily. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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/frame/FrameView.h
diff --git a/third_party/WebKit/Source/core/frame/FrameView.h b/third_party/WebKit/Source/core/frame/FrameView.h
index 2bb1247fa689d8a7f9b102b69ab36d47825cc2ce..f79af59afbf4d8c8f1b5579e98315fc6e20634cd 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.h
+++ b/third_party/WebKit/Source/core/frame/FrameView.h
@@ -127,7 +127,7 @@ public:
bool needsLayout() const;
void setNeedsLayout();
- void setNeedsUpdateWidgetPositions() { m_needsUpdateWidgetPositions = true; }
+ void setNeedsUpdateWidgetGeometries() { m_needsUpdateWidgetGeometries = true; }
// Methods for getting/setting the size Blink should use to layout the contents.
// NOTE: Scrollbar exclusion is based on the FrameView's scrollbars. To exclude
@@ -215,7 +215,7 @@ public:
void addPart(LayoutPart*);
void removePart(LayoutPart*);
- void updateWidgetPositions();
+ void updateWidgetGeometries();
void addPartToUpdate(LayoutEmbeddedObject&);
@@ -677,7 +677,7 @@ private:
IntPoint convertToContainingWidget(const IntPoint&) const override;
IntPoint convertFromContainingWidget(const IntPoint&) const override;
- void updateWidgetPositionsIfNeeded();
+ void updateWidgetGeometriesIfNeeded();
bool wasViewportResized();
void sendResizeEventIfNeeded();
@@ -835,7 +835,7 @@ private:
float m_topControlsViewportAdjustment;
- bool m_needsUpdateWidgetPositions;
+ bool m_needsUpdateWidgetGeometries;
bool m_needsUpdateViewportIntersection;
bool m_needsUpdateViewportIntersectionInSubtree;
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/frame/FrameView.cpp » ('j') | third_party/WebKit/Source/core/frame/FrameView.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698