Add SerializedScriptValueTest.

Add SerializedScriptValueTest.

Committed: https://code.google.com/p/dart/source/detail?r=6530

[podivilov, 2012-04-13 16:23:23]

https://chromiumcodereview.appspot.com/10079008/diff/1/client/tests/client/do...
File client/tests/client/dom/SerializedScriptValueTest.dart (right):

https://chromiumcodereview.appspot.com/10079008/diff/1/client/tests/client/do...
client/tests/client/dom/SerializedScriptValueTest.dart:32: final cyclicList = [
dagList ]; cyclicList.add(cyclicList);
nit: missing new line

https://chromiumcodereview.appspot.com/10079008/diff/1/client/tests/client/do...
client/tests/client/dom/SerializedScriptValueTest.dart:33:
serializationTest('simple list', simpleMap);
Did you mean to use maps here and below?

[Anton Muhin, 2012-04-13 16:25:52]

https://chromiumcodereview.appspot.com/10079008/diff/1/client/tests/client/do...
File client/tests/client/dom/SerializedScriptValueTest.dart (right):

https://chromiumcodereview.appspot.com/10079008/diff/1/client/tests/client/do...
client/tests/client/dom/SerializedScriptValueTest.dart:32: final cyclicList = [
dagList ]; cyclicList.add(cyclicList);
On 2012/04/13 16:23:23, podivilov wrote:
> nit: missing new line

It was intentanional, to show those are closely related.  If you don't like it,
I'll split the line.

https://chromiumcodereview.appspot.com/10079008/diff/1/client/tests/client/do...
client/tests/client/dom/SerializedScriptValueTest.dart:33:
serializationTest('simple list', simpleMap);
On 2012/04/13 16:23:23, podivilov wrote:
> Did you mean to use maps here and below?

Thanks a lot for spotting this.

[podivilov, 2012-04-13 16:26:54]

http://codereview.chromium.org/10079008/diff/1/client/tests/client/dom/Serial...
File client/tests/client/dom/SerializedScriptValueTest.dart (right):

http://codereview.chromium.org/10079008/diff/1/client/tests/client/dom/Serial...
client/tests/client/dom/SerializedScriptValueTest.dart:32: final cyclicList = [
dagList ]; cyclicList.add(cyclicList);
On 2012/04/13 16:25:52, antonmuhin wrote:
> On 2012/04/13 16:23:23, podivilov wrote:
> > nit: missing new line
> 
> It was intentanional, to show those are closely related.  If you don't like
it,
> I'll split the line.

Yes, please.

[Anton Muhin, 2012-04-13 16:28:04]

http://codereview.chromium.org/10079008/diff/1/client/tests/client/dom/Serial...
File client/tests/client/dom/SerializedScriptValueTest.dart (right):

http://codereview.chromium.org/10079008/diff/1/client/tests/client/dom/Serial...
client/tests/client/dom/SerializedScriptValueTest.dart:32: final cyclicList = [
dagList ]; cyclicList.add(cyclicList);
On 2012/04/13 16:26:54, podivilov wrote:
> On 2012/04/13 16:25:52, antonmuhin wrote:
> > On 2012/04/13 16:23:23, podivilov wrote:
> > > nit: missing new line
> > 
> > It was intentanional, to show those are closely related.  If you don't like
> it,
> > I'll split the line.
> 
> Yes, please.

Done.

[podivilov, 2012-04-13 16:30:24]

lgtm