From 398bbb6aa9c17e3db49cb612afcd4ba5e05846ce Mon Sep 17 00:00:00 2001
From: Zhicheng Wang <asnowwolf@gmail.com>
Date: Sat, 13 Aug 2016 07:08:37 +0800
Subject: [PATCH] refactor(core): replace some `for` statements with map/reduce

---
 modules/@angular/compiler/src/shadow_css.ts | 36 ++++++++++-----------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/modules/@angular/compiler/src/shadow_css.ts b/modules/@angular/compiler/src/shadow_css.ts
index 42aa5604c8..f5fa08de52 100644
--- a/modules/@angular/compiler/src/shadow_css.ts
+++ b/modules/@angular/compiler/src/shadow_css.ts
@@ -323,10 +323,8 @@ export class ShadowCss {
    * by replacing with space.
   */
   private _convertShadowDOMSelectors(cssText: string): string {
-    for (let i = 0; i < _shadowDOMSelectorsRe.length; i++) {
-      cssText = StringWrapper.replaceAll(cssText, _shadowDOMSelectorsRe[i], ' ');
-    }
-    return cssText;
+    return _shadowDOMSelectorsRe.reduce(
+        (result, pattern) => { return StringWrapper.replaceAll(result, pattern, ' '); }, cssText);
   }
 
   // change a selector like 'div' to 'name div'
@@ -346,20 +344,22 @@ export class ShadowCss {
 
   private _scopeSelector(
       selector: string, scopeSelector: string, hostSelector: string, strict: boolean): string {
-    const r: any[] /** TODO #9100 */ = [], parts = selector.split(',');
-    for (let i = 0; i < parts.length; i++) {
-      const p = parts[i].trim();
-      const deepParts = StringWrapper.split(p, _shadowDeepSelectors);
-      const shallowPart = deepParts[0];
-      if (this._selectorNeedsScoping(shallowPart, scopeSelector)) {
-        deepParts[0] = strict && !StringWrapper.contains(shallowPart, _polyfillHostNoCombinator) ?
-            this._applyStrictSelectorScope(shallowPart, scopeSelector) :
-            this._applySelectorScope(shallowPart, scopeSelector, hostSelector);
-      }
-      // replace /deep/ with a space for child selectors
-      r.push(deepParts.join(' '));
-    }
-    return r.join(', ');
+    return selector.split(',')
+        .map((part) => { return StringWrapper.split(part.trim(), _shadowDeepSelectors); })
+        .map((deepParts) => {
+          const [shallowPart, ...otherParts] = deepParts;
+          const applyScope = (shallowPart: string) => {
+            if (this._selectorNeedsScoping(shallowPart, scopeSelector)) {
+              return strict && !StringWrapper.contains(shallowPart, _polyfillHostNoCombinator) ?
+                  this._applyStrictSelectorScope(shallowPart, scopeSelector) :
+                  this._applySelectorScope(shallowPart, scopeSelector, hostSelector);
+            } else {
+              return shallowPart;
+            }
+          };
+          return [applyScope(shallowPart), ...otherParts].join(' ');
+        })
+        .join(', ');
   }
 
   private _selectorNeedsScoping(selector: string, scopeSelector: string): boolean {