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

Side by Side Diff: Source/core/rendering/RenderObject.cpp

Issue 131233003: Refactor ResourceLoadPriorityOptimizer to avoid walking render tree (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Use esprehn's suggestion. Created 6 years, 11 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) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2000 Dirk Mueller (mueller@kde.org) 4 * (C) 2000 Dirk Mueller (mueller@kde.org)
5 * (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com) 5 * (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights reserv ed. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights reserv ed.
7 * Copyright (C) 2009 Google Inc. All rights reserved. 7 * Copyright (C) 2009 Google Inc. All rights reserved.
8 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/) 8 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/)
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 2842 matching lines...) Expand 10 before | Expand all | Expand 10 after
2853 } else { 2853 } else {
2854 if (isRooted()) { 2854 if (isRooted()) {
2855 if (RenderView* renderView = view()) { 2855 if (RenderView* renderView = view()) {
2856 if (FrameView* frameView = renderView->frameView()) 2856 if (FrameView* frameView = renderView->frameView())
2857 frameView->scheduleRelayoutOfSubtree(this); 2857 frameView->scheduleRelayoutOfSubtree(this);
2858 } 2858 }
2859 } 2859 }
2860 } 2860 }
2861 } 2861 }
2862 2862
2863 void RenderObject::didLayout(ResourceLoadPriorityOptimizer& priorityModifier)
2864 {
2865 for (RenderObject* child = firstChild(); child; child = child->nextSibling() )
2866 child->didLayout(priorityModifier);
2867 }
2868
2869 void RenderObject::didScroll(ResourceLoadPriorityOptimizer& priorityModifier)
2870 {
2871 for (RenderObject* child = firstChild(); child; child = child->nextSibling() )
2872 child->didScroll(priorityModifier);
2873 }
2874
2875 void RenderObject::forceLayout() 2863 void RenderObject::forceLayout()
2876 { 2864 {
2877 setSelfNeedsLayout(true); 2865 setSelfNeedsLayout(true);
2878 setShouldDoFullRepaintAfterLayout(true); 2866 setShouldDoFullRepaintAfterLayout(true);
2879 layout(); 2867 layout();
2880 } 2868 }
2881 2869
2882 // FIXME: Does this do anything different than forceLayout given that we don't w alk 2870 // FIXME: Does this do anything different than forceLayout given that we don't w alk
2883 // the containing block chain. If not, we should change all callers to use force Layout. 2871 // the containing block chain. If not, we should change all callers to use force Layout.
2884 void RenderObject::forceChildLayout() 2872 void RenderObject::forceChildLayout()
(...skipping 522 matching lines...) Expand 10 before | Expand all | Expand 10 after
3407 { 3395 {
3408 if (object1) { 3396 if (object1) {
3409 const WebCore::RenderObject* root = object1; 3397 const WebCore::RenderObject* root = object1;
3410 while (root->parent()) 3398 while (root->parent())
3411 root = root->parent(); 3399 root = root->parent();
3412 root->showRenderTreeAndMark(object1, "*", object2, "-", 0); 3400 root->showRenderTreeAndMark(object1, "*", object2, "-", 0);
3413 } 3401 }
3414 } 3402 }
3415 3403
3416 #endif 3404 #endif
OLDNEW
« Source/core/rendering/RenderImage.cpp ('K') | « Source/core/rendering/RenderObject.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698