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

Side by Side Diff: Source/core/html/parser/HTMLConstructionSite.cpp

Issue 23475019: The HTML parser should lazy attach. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 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 | Annotate | Revision Log
« no previous file with comments | « LayoutTests/transitions/repeated-firing-background-color-expected.txt ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2010 Google, Inc. All Rights Reserved. 2 * Copyright (C) 2010 Google, Inc. All Rights Reserved.
3 * Copyright (C) 2011 Apple Inc. All rights reserved. 3 * Copyright (C) 2011 Apple Inc. All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. 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 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 if (task.nextChild) 98 if (task.nextChild)
99 task.parent->parserInsertBefore(task.child.get(), task.nextChild.get(), attachBehavior); 99 task.parent->parserInsertBefore(task.child.get(), task.nextChild.get(), attachBehavior);
100 else 100 else
101 task.parent->parserAppendChild(task.child.get(), attachBehavior); 101 task.parent->parserAppendChild(task.child.get(), attachBehavior);
102 } 102 }
103 103
104 static inline void executeInsertTask(HTMLConstructionSiteTask& task) 104 static inline void executeInsertTask(HTMLConstructionSiteTask& task)
105 { 105 {
106 ASSERT(task.operation == HTMLConstructionSiteTask::Insert); 106 ASSERT(task.operation == HTMLConstructionSiteTask::Insert);
107 107
108 insert(task, DeprecatedAttachNow); 108 insert(task);
eseidel 2013/09/04 18:39:14 I assume you meant to include this? :)
ojan 2013/09/04 18:43:59 This is *the* change. Stupid rietveld doesn't upda
109 109
110 task.child->beginParsingChildren(); 110 task.child->beginParsingChildren();
111 111
112 if (task.selfClosing) 112 if (task.selfClosing)
113 task.child->finishParsingChildren(); 113 task.child->finishParsingChildren();
114 } 114 }
115 115
116 static inline void executeReparentTask(HTMLConstructionSiteTask& task) 116 static inline void executeReparentTask(HTMLConstructionSiteTask& task)
117 { 117 {
118 ASSERT(task.operation == HTMLConstructionSiteTask::Reparent); 118 ASSERT(task.operation == HTMLConstructionSiteTask::Reparent);
(...skipping 625 matching lines...) Expand 10 before | Expand all | Expand 10 after
744 { 744 {
745 HTMLConstructionSiteTask task(HTMLConstructionSiteTask::Insert); 745 HTMLConstructionSiteTask task(HTMLConstructionSiteTask::Insert);
746 findFosterSite(task); 746 findFosterSite(task);
747 task.child = node; 747 task.child = node;
748 ASSERT(task.parent); 748 ASSERT(task.parent);
749 749
750 m_taskQueue.append(task); 750 m_taskQueue.append(task);
751 } 751 }
752 752
753 } 753 }
OLDNEW
« no previous file with comments | « LayoutTests/transitions/repeated-firing-background-color-expected.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698