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

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

Issue 2657743004: Handle stale m_owner in FrameView::needsScrollbarReconstruction. (Closed)
Patch Set: rebase Created 3 years, 10 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 7 Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights
8 reserved. 8 reserved.
9 9
10 This library is free software; you can redistribute it and/or 10 This library is free software; you can redistribute it and/or
(...skipping 961 matching lines...) Expand 10 before | Expand all | Expand 10 after
972 // Returns true if the FrameView's own scrollbars overlay its content when 972 // Returns true if the FrameView's own scrollbars overlay its content when
973 // visible. 973 // visible.
974 bool hasOverlayScrollbars() const; 974 bool hasOverlayScrollbars() const;
975 975
976 // Returns true if the frame should use custom scrollbars. If true, sets 976 // Returns true if the frame should use custom scrollbars. If true, sets
977 // customScrollbarElement to the element that supplies the scrollbar's style 977 // customScrollbarElement to the element that supplies the scrollbar's style
978 // information. 978 // information.
979 bool shouldUseCustomScrollbars(Element*& customScrollbarElement) const; 979 bool shouldUseCustomScrollbars(Element*& customScrollbarElement) const;
980 980
981 // Returns true if a scrollbar needs to go from native -> custom or vice 981 // Returns true if a scrollbar needs to go from native -> custom or vice
982 // versa. 982 // versa, or if a custom scrollbar has a stale owner.
983 bool needsScrollbarReconstruction() const; 983 bool needsScrollbarReconstruction() const;
984 984
985 bool shouldIgnoreOverflowHidden() const; 985 bool shouldIgnoreOverflowHidden() const;
986 986
987 void updateScrollCorner(); 987 void updateScrollCorner();
988 988
989 AXObjectCache* axObjectCache() const; 989 AXObjectCache* axObjectCache() const;
990 990
991 void setLayoutSizeInternal(const IntSize&); 991 void setLayoutSizeInternal(const IntSize&);
992 992
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
1241 widget.isFrameView()); 1241 widget.isFrameView());
1242 DEFINE_TYPE_CASTS(FrameView, 1242 DEFINE_TYPE_CASTS(FrameView,
1243 ScrollableArea, 1243 ScrollableArea,
1244 scrollableArea, 1244 scrollableArea,
1245 scrollableArea->isFrameView(), 1245 scrollableArea->isFrameView(),
1246 scrollableArea.isFrameView()); 1246 scrollableArea.isFrameView());
1247 1247
1248 } // namespace blink 1248 } // namespace blink
1249 1249
1250 #endif // FrameView_h 1250 #endif // FrameView_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698