From 5b29fb79b4367a0dd6ec393d39c1df1828db4588 Mon Sep 17 00:00:00 2001
From: Rafael Weinstein <rafaelw@chromium.org>
Date: Thu, 9 May 2013 13:53:58 -0700
Subject: [PATCH] Use ArrayReduction.defineProperty

R=arv
BUG=

Review URL: https://codereview.appspot.com/9333043
---
 examples/how_to/array_reduction.html | 29 ++++++++++++++++++++--------
 third_party/ChangeSummary            |  2 +-
 2 files changed, 22 insertions(+), 9 deletions(-)

diff --git a/examples/how_to/array_reduction.html b/examples/how_to/array_reduction.html
index 1cc8ab3..dfb1b20 100644
--- a/examples/how_to/array_reduction.html
+++ b/examples/how_to/array_reduction.html
@@ -7,24 +7,37 @@ <h1>Reduction</h1>
   <template repeat="{{ items }}">
     <div>{{ name }}: <input type="checkbox" checked="{{ selected }}"><input type="number" value="{{ amount }}"></div>
   </template>
-  Sum: <input disabled value="{{ sum.value }}"><button disabled?="{{ noneSelected.value }}">Only Enable If Any Selected</button>
+  Sum: <input disabled value="{{ sum }}"><button disabled?="{{ noneSelected }}">Only Enable If Any Selected</button>
 </template>
 </form>
 
 <script>
-var t = document.getElementById('reduction');
-t.model = {
+var model = {
   items: [
     { name: 'One', amount: 1, selected: false },
     { name: 'Two', amount: 2, selected: false },
     { name: 'Three', amount: 3, selected: false }
   ]
 };
-t.model.noneSelected = new ArrayReduction(t.model.items, 'selected', function(prev, current) {
-  return prev && !current;
-}, true);
 
-t.model.sum = new ArrayReduction(t.model.items, 'amount', function(prev, current) {
-  return Number(prev) + Number(current);
+// reduction1 must be close()'d if Object.observe is not implemented otherwise => memory leak.
+var reduction1 = ArrayReduction.defineProperty(model, 'noneSelected', {
+  array: model.items,
+  path: 'selected',
+  initial: true,
+  reduce: function(prev, current) {
+    return prev && !current;
+  }
 });
+
+// reduction2 must be close()'d if Object.observe is not implemented otherwise => memory leak.
+var reduction2 = ArrayReduction.defineProperty(model, 'sum', {
+  array: model.items,
+  path: 'amount',
+  reduce: function(prev, current) {
+    return Number(prev) + Number(current);
+  }
+});
+
+document.getElementById('reduction').model = model;
 </script>
diff --git a/third_party/ChangeSummary b/third_party/ChangeSummary
index 6d333c9..454d61e 160000
--- a/third_party/ChangeSummary
+++ b/third_party/ChangeSummary
@@ -1 +1 @@
-Subproject commit 6d333c9b8572f1880e88737bb85284a9ded3b2f6
+Subproject commit 454d61ec658d2ccdea55c8a5931799d8ad1f750c