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

Side by Side Diff: Source/core/frame/FrameView.cpp

Issue 503583002: Fixed Inconsistent behaviour of custom-scrollbar (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Adding test case Created 6 years, 3 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) 1998, 1999 Torben Weis <weis@kde.org> 2 * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
3 * 1999 Lars Knoll <knoll@kde.org> 3 * 1999 Lars Knoll <knoll@kde.org>
4 * 1999 Antti Koivisto <koivisto@kde.org> 4 * 1999 Antti Koivisto <koivisto@kde.org>
5 * 2000 Dirk Mueller <mueller@kde.org> 5 * 2000 Dirk Mueller <mueller@kde.org>
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
7 * (C) 2006 Graham Dennis (graham.dennis@gmail.com) 7 * (C) 2006 Graham Dennis (graham.dennis@gmail.com)
8 * (C) 2006 Alexey Proskuryakov (ap@nypop.com) 8 * (C) 2006 Alexey Proskuryakov (ap@nypop.com)
9 * Copyright (C) 2009 Google Inc. All rights reserved. 9 * Copyright (C) 2009 Google Inc. All rights reserved.
10 * 10 *
(...skipping 902 matching lines...) Expand 10 before | Expand all | Expand 10 after
913 // We need to ensure that we mark up all renderers up to the RenderView 913 // We need to ensure that we mark up all renderers up to the RenderView
914 // for paint invalidation. This simplifies our code as we just always 914 // for paint invalidation. This simplifies our code as we just always
915 // do a full tree walk. 915 // do a full tree walk.
916 if (RenderObject* container = rootForThisLayout->container()) 916 if (RenderObject* container = rootForThisLayout->container())
917 container->setMayNeedPaintInvalidation(true); 917 container->setMayNeedPaintInvalidation(true);
918 } // Reset m_layoutSchedulingEnabled to its previous value. 918 } // Reset m_layoutSchedulingEnabled to its previous value.
919 919
920 if (!inSubtreeLayout && !toRenderView(rootForThisLayout)->document().printin g()) 920 if (!inSubtreeLayout && !toRenderView(rootForThisLayout)->document().printin g())
921 adjustViewSize(); 921 adjustViewSize();
922 922
923 if (rootForThisLayout->needsLayout())
924 performLayout(rootForThisLayout, inSubtreeLayout);
esprehn 2014/09/19 05:06:52 Other places call adjustViewSize without a second
MuVen 2014/09/19 16:19:30 During adjustViewSize call, in the API FrameView::
925
923 layer->updateLayerPositionsAfterLayout(); 926 layer->updateLayerPositionsAfterLayout();
924 927
925 if (m_doFullPaintInvalidation) 928 if (m_doFullPaintInvalidation)
926 renderView()->compositor()->fullyInvalidatePaint(); 929 renderView()->compositor()->fullyInvalidatePaint();
927 renderView()->compositor()->didLayout(); 930 renderView()->compositor()->didLayout();
928 931
929 m_layoutCount++; 932 m_layoutCount++;
930 933
931 if (AXObjectCache* cache = rootForThisLayout->document().axObjectCache()) { 934 if (AXObjectCache* cache = rootForThisLayout->document().axObjectCache()) {
932 const KURL& url = rootForThisLayout->document().url(); 935 const KURL& url = rootForThisLayout->document().url();
(...skipping 2043 matching lines...) Expand 10 before | Expand all | Expand 10 after
2976 void FrameView::willRemoveScrollbar(Scrollbar* scrollbar, ScrollbarOrientation o rientation) 2979 void FrameView::willRemoveScrollbar(Scrollbar* scrollbar, ScrollbarOrientation o rientation)
2977 { 2980 {
2978 ScrollableArea::willRemoveScrollbar(scrollbar, orientation); 2981 ScrollableArea::willRemoveScrollbar(scrollbar, orientation);
2979 if (AXObjectCache* cache = axObjectCache()) { 2982 if (AXObjectCache* cache = axObjectCache()) {
2980 cache->remove(scrollbar); 2983 cache->remove(scrollbar);
2981 cache->handleScrollbarUpdate(this); 2984 cache->handleScrollbarUpdate(this);
2982 } 2985 }
2983 } 2986 }
2984 2987
2985 } // namespace blink 2988 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698