honeymoose
/
angular-cn
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

refactor(ivy): misc cleanup (#23441) 

PR Close #23441
This commit is contained in:
Victor Berchet 2018-04-16 11:49:11 -07:00
parent c6b206ee4b
commit 84f024309a
2 changed files with 20 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
packages/compiler/src/render3/r3_view_compiler.ts
View File

@ -575,16 +575,21 @@ class TemplateDefinitionBuilder implements TemplateAstVisitor, LocalResolver {
// TemplateAstVisitor // TemplateAstVisitor
visitNgContent(ngContent: NgContentAst) { visitNgContent(ngContent: NgContentAst) {
const info = this._contentProjections.get(ngContent) !; const info = this._contentProjections.get(ngContent);
info || if (!info) {
error(`Expected ${ngContent.sourceSpan} to be included in content projection collection`); error(`Expected ${ngContent.sourceSpan} to be included in content projection collection`);
} else {
const slot = this.allocateDataSlot(); const slot = this.allocateDataSlot();
const parameters = [o.literal(slot), o.literal(this._projectionDefinitionIndex)]; const parameters = [
o.literal(slot),
o.literal(this._projectionDefinitionIndex),
];
if (info.index !== 0) { if (info.index !== 0) {
parameters.push(o.literal(info.index)); parameters.push(o.literal(info.index));
} }
this.instruction(this._creationMode, ngContent.sourceSpan, R3.projection, ...parameters); this.instruction(this._creationMode, ngContent.sourceSpan, R3.projection, ...parameters);
} }
}
// TemplateAstVisitor // TemplateAstVisitor
visitElement(element: ElementAst) { visitElement(element: ElementAst) {

18
packages/core/src/render3/instructions.ts
View File

@ -1722,32 +1722,28 @@ function appendToProjectionNode(
* @param nodeIndex * @param nodeIndex
* @param localIndex - index under which distribution of projected nodes was memorized * @param localIndex - index under which distribution of projected nodes was memorized
* @param selectorIndex - 0 means <ng-content> without any selector * @param selectorIndex - 0 means <ng-content> without any selector
* @param attrs - attributes attached to the ng-content node, if present
*/ */
export function projection( export function projection(nodeIndex: number, localIndex: number, selectorIndex: number = 0): void {
nodeIndex: number, localIndex: number, selectorIndex: number = 0, attrs?: string[]): void {
const node = createLNode(nodeIndex, LNodeType.Projection, null, {head: null, tail: null}); const node = createLNode(nodeIndex, LNodeType.Projection, null, {head: null, tail: null});
if (node.tNode == null) { // `<ng-content>` has no content
node.tNode = createTNode(null, attrs || null, null); isParent = false;
}
isParent = false; // self closing
const currentParent = node.parent; const currentParent = node.parent;
// re-distribution of projectable nodes is memorized on a component's view level // re-distribution of projectable nodes is memorized on a component's view level
const componentNode = findComponentHost(currentView); const componentNode = findComponentHost(currentView);
const componentLView = componentNode.data !;
// make sure that nodes to project were memorized const nodesForSelector = componentLView.data ![localIndex][selectorIndex];
const nodesForSelector = componentNode.data !.data ![localIndex][selectorIndex];
// build the linked list of projected nodes: // build the linked list of projected nodes:
for (let i = 0; i < nodesForSelector.length; i++) { for (let i = 0; i < nodesForSelector.length; i++) {
const nodeToProject = nodesForSelector[i]; const nodeToProject = nodesForSelector[i];
if (nodeToProject.type === LNodeType.Projection) { if (nodeToProject.type === LNodeType.Projection) {
// Reprojecting a projection -> append the list of previously projected nodes
const previouslyProjected = (nodeToProject as LProjectionNode).data; const previouslyProjected = (nodeToProject as LProjectionNode).data;
appendToProjectionNode(node, previouslyProjected.head, previouslyProjected.tail); appendToProjectionNode(node, previouslyProjected.head, previouslyProjected.tail);
} else { } else {
// Projecting a single node
appendToProjectionNode( appendToProjectionNode(
node, nodeToProject as LTextNode | LElementNode | LContainerNode, node, nodeToProject as LTextNode | LElementNode | LContainerNode,
nodeToProject as LTextNode | LElementNode | LContainerNode); nodeToProject as LTextNode | LElementNode | LContainerNode);