fix bug with rootObj optimization in observed sets

src/observe.js

@@ -390,7 +390,7 @@
           obj = obj[this[i - 1]];
         if (!isObject(obj))
           return;
-        observe(obj, this[0]);
+        observe(obj, this[i]);
       }
     },
 
@@ -663,14 +663,13 @@
     }
 
     var record = {
-      object: undefined,
       objects: objects,
+      get rootObject() { return rootObj; },
+      set rootObject(value) {
+        rootObj = value;
+        rootObjProps = {};
+      },
       open: function(obs, object) {
-        if (!rootObj) {
-          rootObj = object;
-          rootObjProps = {};
-        }
-
         observers.push(obs);
         observerCount++;
         obs.iterateObjects_(observe);
@@ -691,7 +690,9 @@
         rootObj = undefined;
         rootObjProps = undefined;
         observedSetCache.push(this);
-      }
+        if (lastObservedSet === this)
+          lastObservedSet = null;
+      },
     };
 
     return record;
@@ -700,9 +701,9 @@
   var lastObservedSet;
 
   function getObservedSet(observer, obj) {
-    if (!lastObservedSet || lastObservedSet.object !== obj) {
+    if (!lastObservedSet || lastObservedSet.rootObject !== obj) {
       lastObservedSet = observedSetCache.pop() || newObservedSet();
-      lastObservedSet.object = obj;
+      lastObservedSet.rootObject = obj;
     }
     lastObservedSet.open(observer, obj);
     return lastObservedSet;

tests/test.js

@@ -942,6 +942,26 @@ suite('PathObserver Tests', function() {
       done();
     });
   });
+
+  test('object cycle', function(done) {
+    var model = { a: {}, c: 1 };
+    model.a.b = model;
+
+    var called = 0;
+    new PathObserver(model, 'a.b.c').open(function() {
+      called++;
+    });
+
+    // This change should be detected, even though it's a change to the root
+    // object and isn't a change to `a`.
+    model.c = 42;
+
+    then(function() {
+      assert.equal(called, 1);
+      done();
+    });
+  });
+
 });