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

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

Issue 1996363003: Avoid calling updateAllLifecyclePhasesExceptPaint in SVGImage if not needed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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: 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 c46ee1d490b23307a6b1499050d4eef94e1c7fe4..87bb69c2b7529cd609808cca3c6ba869774bb794 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.h
+++ b/third_party/WebKit/Source/core/frame/FrameView.h
@@ -276,7 +276,8 @@ public:
// If |UrlFragmentScroll| is passed in then makes the anchor element
// focused and also visible by scrolling to it. The scroll position is
// maintained during the frame loading process.
- void processUrlFragment(const KURL&, UrlFragmentBehavior = UrlFragmentScroll);
+ // Returns false if the URL behavior is unchanged from last time this method was called.
fs 2016/05/23 21:25:56 Maybe also mention that false is returned if the f
+ bool processUrlFragment(const KURL&, UrlFragmentBehavior = UrlFragmentScroll);
void clearFragmentAnchor();
// Methods to convert points and rects between the coordinate space of the layoutObject, and this view.
@@ -912,6 +913,9 @@ private:
ScrollAnchor m_scrollAnchor;
bool m_needsScrollbarsUpdate;
+
+ KURL m_currentUrl;
+ UrlFragmentBehavior m_currentUrlFragmentBehavior;
};
inline void FrameView::incrementVisuallyNonEmptyCharacterCount(unsigned count)
« 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