https://codereview.chromium.org/15504002/diff/33001/test/mjsunit/harmony/object-observe.js
File test/mjsunit/harmony/object-observe.js (right):

https://codereview.chromium.org/15504002/diff/33001/test/mjsunit/harmony/object-observe.js#newcode1005
test/mjsunit/harmony/object-observe.js:1005: { object: arr3, name:
'length', type: 'reconfigured' }
On 2013/06/04 10:56:58, rossberg wrote:
On 2013/05/28 11:38:53, rafaelw wrote:
> It isn't correct, but Adam and I discussed and decided that
>
> a) It's fine to break with spec compliance here because it's an
extreme edge
> case and even if hit, two records vs one is almost certain not to
confuse any
> observer.
>
> b) Making it spec compliant in this minor way isn't worth layering
hacks on
top
> of more hacks here. Once the todo mentioned by mstarzinger is fixed,
the right
> change record sequence will return.
>
> On 2013/05/27 14:48:41, rossberg wrote:
> > Not sure I understand, why is it correct that this a separate
change now?
>

OK, can you add a TODO then?

Done.

https://codereview.chromium.org/15504002/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to