Chromium Code Reviews| OLD | NEW |
|---|---|
| 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) 2001 Dirk Mueller (mueller@kde.org) | 4 * (C) 2001 Dirk Mueller (mueller@kde.org) |
| 5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ights reserved. | 5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ights reserved. |
| 6 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) | 6 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) |
| 7 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/) | 7 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/) |
| 8 * | 8 * |
| 9 * This library is free software; you can redistribute it and/or | 9 * This library is free software; you can redistribute it and/or |
| 10 * modify it under the terms of the GNU Library General Public | 10 * modify it under the terms of the GNU Library General Public |
| (...skipping 756 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 767 document().scheduleRenderTreeUpdateIfNeeded(); | 767 document().scheduleRenderTreeUpdateIfNeeded(); |
| 768 } | 768 } |
| 769 | 769 |
| 770 void Node::setNeedsStyleRecalc(StyleChangeType changeType, const StyleChangeReas onForTracing& reason) | 770 void Node::setNeedsStyleRecalc(StyleChangeType changeType, const StyleChangeReas onForTracing& reason) |
| 771 { | 771 { |
| 772 ASSERT(changeType != NoStyleChange); | 772 ASSERT(changeType != NoStyleChange); |
| 773 if (!inActiveDocument()) | 773 if (!inActiveDocument()) |
| 774 return; | 774 return; |
| 775 | 775 |
| 776 StyleChangeType existingChangeType = styleChangeType(); | 776 StyleChangeType existingChangeType = styleChangeType(); |
| 777 if (changeType > existingChangeType) { | 777 if (changeType > existingChangeType) { |
|
pdr.
2014/09/30 01:51:20
We may want to do this for all changes, not just o
kouhei (in TOK)
2014/09/30 02:01:54
Done.
| |
| 778 setStyleChange(changeType); | 778 setStyleChange(changeType); |
| 779 if (changeType >= SubtreeStyleChange) | 779 if (changeType >= SubtreeStyleChange) { |
|
pdr.
2014/09/30 01:51:20
Is this correct? What about LocalStyleChanges?
kouhei (in TOK)
2014/09/30 02:01:54
Done.
| |
| 780 TRACE_EVENT_INSTANT1( | |
| 781 TRACE_DISABLED_BY_DEFAULT("devtools.timeline.invalidationTrackin g"), | |
| 782 "StyleRecalcInvalidationTracking", | |
| 783 "data", | |
| 784 InspectorStyleRecalcInvalidationTrackingEvent::data(this, reason )); | |
| 785 | |
| 786 // FIXME: Remove below | |
|
pdr.
2014/09/30 01:51:20
Fixed! https://src.chromium.org/viewvc/blink?revis
kouhei (in TOK)
2014/09/30 02:01:54
Thanks!
| |
| 780 traceStyleChangeIfNeeded(changeType); | 787 traceStyleChangeIfNeeded(changeType); |
| 788 } | |
| 781 } | 789 } |
| 782 | 790 |
| 783 if (existingChangeType == NoStyleChange) | 791 if (existingChangeType == NoStyleChange) |
| 784 markAncestorsWithChildNeedsStyleRecalc(); | 792 markAncestorsWithChildNeedsStyleRecalc(); |
| 785 | 793 |
| 786 if (isElementNode() && hasRareData()) | 794 if (isElementNode() && hasRareData()) |
| 787 toElement(*this).setAnimationStyleChange(false); | 795 toElement(*this).setAnimationStyleChange(false); |
| 788 } | 796 } |
| 789 | 797 |
| 790 void Node::clearNeedsStyleRecalc() | 798 void Node::clearNeedsStyleRecalc() |
| (...skipping 1719 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 2510 node->showTreeForThis(); | 2518 node->showTreeForThis(); |
| 2511 } | 2519 } |
| 2512 | 2520 |
| 2513 void showNodePath(const blink::Node* node) | 2521 void showNodePath(const blink::Node* node) |
| 2514 { | 2522 { |
| 2515 if (node) | 2523 if (node) |
| 2516 node->showNodePathForThis(); | 2524 node->showNodePathForThis(); |
| 2517 } | 2525 } |
| 2518 | 2526 |
| 2519 #endif | 2527 #endif |
| OLD | NEW |