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

Side by Side Diff: third_party/WebKit/Source/core/frame/FrameView.h

Issue 2120773002: Simplify and solidify FrameView dirty layout check (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: FrameView dirty layout check Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 Copyright (C) 1997 Martin Jones (mjones@kde.org) 2 Copyright (C) 1997 Martin Jones (mjones@kde.org)
3 (C) 1998 Waldo Bastian (bastian@kde.org) 3 (C) 1998 Waldo Bastian (bastian@kde.org)
4 (C) 1998, 1999 Torben Weis (weis@kde.org) 4 (C) 1998, 1999 Torben Weis (weis@kde.org)
5 (C) 1999 Lars Knoll (knoll@kde.org) 5 (C) 1999 Lars Knoll (knoll@kde.org)
6 (C) 1999 Antti Koivisto (koivisto@kde.org) 6 (C) 1999 Antti Koivisto (koivisto@kde.org)
7 Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserv ed. 7 Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserv ed.
8 8
9 This library is free software; you can redistribute it and/or 9 This library is free software; you can redistribute it and/or
10 modify it under the terms of the GNU Library General Public 10 modify it under the terms of the GNU Library General Public
(...skipping 895 matching lines...) Expand 10 before | Expand all | Expand 10 after
906 // TODO(trchen): This will not be needed once settings->rootLayerScrolls() i s enabled. 906 // TODO(trchen): This will not be needed once settings->rootLayerScrolls() i s enabled.
907 RefPtr<ClipPaintPropertyNode> m_contentClip; 907 RefPtr<ClipPaintPropertyNode> m_contentClip;
908 908
909 // This is set on the local root frame view only. 909 // This is set on the local root frame view only.
910 DocumentLifecycle::LifecycleState m_currentUpdateLifecyclePhasesTargetState; 910 DocumentLifecycle::LifecycleState m_currentUpdateLifecyclePhasesTargetState;
911 911
912 ScrollAnchor m_scrollAnchor; 912 ScrollAnchor m_scrollAnchor;
913 913
914 bool m_needsScrollbarsUpdate; 914 bool m_needsScrollbarsUpdate;
915 bool m_suppressAdjustViewSize; 915 bool m_suppressAdjustViewSize;
916 bool m_inPluginUpdate;
917 bool m_inForcedLayoutByChildEmbeddedReplacedContent;
918 bool m_allowsLayoutInvalidationAfterLayoutClean; 916 bool m_allowsLayoutInvalidationAfterLayoutClean;
919 917
920 // For testing. 918 // For testing.
921 struct ObjectPaintInvalidation { 919 struct ObjectPaintInvalidation {
922 String name; 920 String name;
923 PaintInvalidationReason reason; 921 PaintInvalidationReason reason;
924 }; 922 };
925 std::unique_ptr<Vector<ObjectPaintInvalidation>> m_trackedObjectPaintInvalid ations; 923 std::unique_ptr<Vector<ObjectPaintInvalidation>> m_trackedObjectPaintInvalid ations;
926 }; 924 };
927 925
(...skipping 19 matching lines...) Expand all
947 if (m_visuallyNonEmptyPixelCount > visualPixelThreshold) 945 if (m_visuallyNonEmptyPixelCount > visualPixelThreshold)
948 setIsVisuallyNonEmpty(); 946 setIsVisuallyNonEmpty();
949 } 947 }
950 948
951 DEFINE_TYPE_CASTS(FrameView, Widget, widget, widget->isFrameView(), widget.isFra meView()); 949 DEFINE_TYPE_CASTS(FrameView, Widget, widget, widget->isFrameView(), widget.isFra meView());
952 DEFINE_TYPE_CASTS(FrameView, ScrollableArea, scrollableArea, scrollableArea->isF rameView(), scrollableArea.isFrameView()); 950 DEFINE_TYPE_CASTS(FrameView, ScrollableArea, scrollableArea, scrollableArea->isF rameView(), scrollableArea.isFrameView());
953 951
954 } // namespace blink 952 } // namespace blink
955 953
956 #endif // FrameView_h 954 #endif // FrameView_h
OLDNEW
« 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