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

Side by Side Diff: third_party/WebKit/Source/core/layout/LayoutObject.cpp

Issue 2754723004: Replace RELEASE_ASSERT with CHECK in core/layout/ (Closed)
Patch Set: Created 3 years, 9 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. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc.
7 * All rights reserved. 7 * All rights reserved.
8 * Copyright (C) 2009 Google Inc. All rights reserved. 8 * Copyright (C) 2009 Google Inc. All rights reserved.
9 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. 9 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved.
10 * (http://www.torchmobile.com/) 10 * (http://www.torchmobile.com/)
(...skipping 2829 matching lines...) Expand 10 before | Expand all | Expand 10 after
2840 continue; // Don't descend into inner fragmentation contexts. 2840 continue; // Don't descend into inner fragmentation contexts.
2841 child->removeFromLayoutFlowThreadRecursive(child->isLayoutFlowThread() 2841 child->removeFromLayoutFlowThreadRecursive(child->isLayoutFlowThread()
2842 ? toLayoutFlowThread(child) 2842 ? toLayoutFlowThread(child)
2843 : layoutFlowThread); 2843 : layoutFlowThread);
2844 } 2844 }
2845 } 2845 }
2846 2846
2847 if (layoutFlowThread && layoutFlowThread != this) 2847 if (layoutFlowThread && layoutFlowThread != this)
2848 layoutFlowThread->flowThreadDescendantWillBeRemoved(this); 2848 layoutFlowThread->flowThreadDescendantWillBeRemoved(this);
2849 setIsInsideFlowThread(false); 2849 setIsInsideFlowThread(false);
2850 RELEASE_ASSERT(!spannerPlaceholder()); 2850 CHECK(!spannerPlaceholder());
2851 } 2851 }
2852 2852
2853 void LayoutObject::destroyAndCleanupAnonymousWrappers() { 2853 void LayoutObject::destroyAndCleanupAnonymousWrappers() {
2854 // If the tree is destroyed, there is no need for a clean-up phase. 2854 // If the tree is destroyed, there is no need for a clean-up phase.
2855 if (documentBeingDestroyed()) { 2855 if (documentBeingDestroyed()) {
2856 destroy(); 2856 destroy();
2857 return; 2857 return;
2858 } 2858 }
2859 2859
2860 LayoutObject* destroyRoot = this; 2860 LayoutObject* destroyRoot = this;
(...skipping 736 matching lines...) Expand 10 before | Expand all | Expand 10 after
3597 const blink::LayoutObject* root = object1; 3597 const blink::LayoutObject* root = object1;
3598 while (root->parent()) 3598 while (root->parent())
3599 root = root->parent(); 3599 root = root->parent();
3600 root->showLayoutTreeAndMark(object1, "*", object2, "-", 0); 3600 root->showLayoutTreeAndMark(object1, "*", object2, "-", 0);
3601 } else { 3601 } else {
3602 WTFLogAlways("%s", "Cannot showLayoutTree. Root is (nil)"); 3602 WTFLogAlways("%s", "Cannot showLayoutTree. Root is (nil)");
3603 } 3603 }
3604 } 3604 }
3605 3605
3606 #endif 3606 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutBox.cpp ('k') | third_party/WebKit/Source/core/layout/LayoutTableSection.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698