Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2013 Google Inc. All rights reserved. | 2 * Copyright (C) 2013 Google Inc. All rights reserved. |
| 3 * | 3 * |
| 4 * Redistribution and use in source and binary forms, with or without | 4 * Redistribution and use in source and binary forms, with or without |
| 5 * modification, are permitted provided that the following conditions are | 5 * modification, are permitted provided that the following conditions are |
| 6 * met: | 6 * met: |
| 7 * | 7 * |
| 8 * * Redistributions of source code must retain the above copyright | 8 * * Redistributions of source code must retain the above copyright |
| 9 * notice, this list of conditions and the following disclaimer. | 9 * notice, this list of conditions and the following disclaimer. |
| 10 * * Redistributions in binary form must reproduce the above | 10 * * Redistributions in binary form must reproduce the above |
| (...skipping 26 matching lines...) Expand all Loading... | |
| 37 namespace WebCore { | 37 namespace WebCore { |
| 38 | 38 |
| 39 SubtreeLayoutScope::SubtreeLayoutScope(RenderObject* root) | 39 SubtreeLayoutScope::SubtreeLayoutScope(RenderObject* root) |
| 40 : m_root(root) | 40 : m_root(root) |
| 41 { | 41 { |
| 42 RELEASE_ASSERT(m_root->document()->view()->isInLayout()); | 42 RELEASE_ASSERT(m_root->document()->view()->isInLayout()); |
| 43 } | 43 } |
| 44 | 44 |
| 45 SubtreeLayoutScope::~SubtreeLayoutScope() | 45 SubtreeLayoutScope::~SubtreeLayoutScope() |
| 46 { | 46 { |
| 47 if (m_root->document()->view()->shouldStopPartialLayout()) | |
|
eseidel
2013/08/20 21:00:39
You might leave a comment here explaining why.
pdr.
2013/08/26 05:50:40
Done, and added an assert that the root does need
| |
| 48 return; | |
| 49 | |
| 47 RELEASE_ASSERT(!m_root->needsLayout()); | 50 RELEASE_ASSERT(!m_root->needsLayout()); |
| 48 | 51 |
| 49 #ifndef NDEBUG | 52 #ifndef NDEBUG |
| 50 for (HashSet<RenderObject*>::iterator it = m_renderersToLayout.begin(); it ! = m_renderersToLayout.end(); ++it) { | 53 for (HashSet<RenderObject*>::iterator it = m_renderersToLayout.begin(); it ! = m_renderersToLayout.end(); ++it) { |
| 51 RenderObject* renderer = *it; | 54 RenderObject* renderer = *it; |
| 52 // FIXME: Thie patter is really common. Move it into an assertRendererLa idOut function. | 55 // FIXME: Thie patter is really common. Move it into an assertRendererLa idOut function. |
| 53 if (renderer->needsLayout()) | 56 if (renderer->needsLayout()) |
| 54 showRenderTree(renderer); | 57 showRenderTree(renderer); |
| 55 ASSERT(!renderer->needsLayout()); | 58 ASSERT(!renderer->needsLayout()); |
| 56 } | 59 } |
| (...skipping 13 matching lines...) Expand all Loading... | |
| 70 } | 73 } |
| 71 | 74 |
| 72 void SubtreeLayoutScope::addRendererToLayout(RenderObject* renderer) | 75 void SubtreeLayoutScope::addRendererToLayout(RenderObject* renderer) |
| 73 { | 76 { |
| 74 #ifndef NDEBUG | 77 #ifndef NDEBUG |
| 75 m_renderersToLayout.add(renderer); | 78 m_renderersToLayout.add(renderer); |
| 76 #endif | 79 #endif |
| 77 } | 80 } |
| 78 | 81 |
| 79 } | 82 } |
| OLD | NEW |