Support default values for complex objects in mojom.

Support default values for complex objects in mojom.

BUG=320082
R=darin@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248579

[Matt Perry, 2014-01-23 19:29:42]

Based on https://codereview.chromium.org/141893004/

[Matt Perry, 2014-01-28 20:34:54]

ping

[darin (slow to review), 2014-01-29 17:51:30]

LGTM

[commit-bot: I haz the power, 2014-01-29 19:24:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mpcomplete@chromium.org/139613005/450001

[commit-bot: I haz the power, 2014-01-29 19:24:22]

Failed to apply patch for
mojo/public/bindings/sample/sample_service_unittests.js:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  A         mojo/public/bindings/sample
  Created missing directory mojo/public/bindings/sample.
  can't find file to patch at input line 6
  Perhaps you used the wrong -p or --strip option?
  The text leading up to this was:
  --------------------------
  |Index: mojo/public/bindings/sample/sample_service_unittests.js
  |diff --git a/mojo/public/bindings/sample/sample_service_unittests.js
b/mojo/public/bindings/sample/sample_service_unittests.js
  |index
9b9165df64e06a1351a85b7b71732ecc3369d111..6e438b3864703b5f8884c368791b2eb88a54654a
100644
  |--- a/mojo/public/bindings/sample/sample_service_unittests.js
  |+++ b/mojo/public/bindings/sample/sample_service_unittests.js
  --------------------------
  No file to patch.  Skipping patch.
  1 out of 1 hunk ignored

Patch:       mojo/public/bindings/sample/sample_service_unittests.js
Index: mojo/public/bindings/sample/sample_service_unittests.js
diff --git a/mojo/public/bindings/sample/sample_service_unittests.js
b/mojo/public/bindings/sample/sample_service_unittests.js
index
9b9165df64e06a1351a85b7b71732ecc3369d111..6e438b3864703b5f8884c368791b2eb88a54654a
100644
--- a/mojo/public/bindings/sample/sample_service_unittests.js
+++ b/mojo/public/bindings/sample/sample_service_unittests.js
@@ -98,6 +98,19 @@ define([
     expect(foo.data[0]).toBe(1);
     expect(foo.data[1]).toBe(2);
     expect(foo.data[2]).toBe(3);
+
+    var inner = new sample.DefaultsTestInner();
+    expect(inner.names.length).toBe(1);
+    expect(inner.names[0]).toBe("Jim");
+    expect(inner.height).toBe(6*12);
+
+    var full = new sample.DefaultsTest();
+    expect(full.people.length).toBe(1);
+    expect(full.people[0].age).toBe(32);
+    expect(full.people[0].names.length).toBe(2);
+    expect(full.people[0].names[0]).toBe("Bob");
+    expect(full.people[0].names[1]).toBe("Bobby");
+    expect(full.people[0].height).toBe(6*12);
   }
 
   function ServiceImpl() {

[commit-bot: I haz the power, 2014-01-29 19:32:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mpcomplete@chromium.org/139613005/470001

[commit-bot: I haz the power, 2014-01-29 20:03:43]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[Matt Perry, 2014-02-03 21:18:10]

The CQ bit was checked by mpcomplete@chromium.org

[commit-bot: I haz the power, 2014-02-03 21:18:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mpcomplete@chromium.org/139613005/490001

[commit-bot: I haz the power, 2014-02-03 21:37:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-03 21:37:16]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2014-02-03 21:37:22]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-03 21:37:26]

CQ bit was unchecked on CL. Ignoring.

[Matt Perry, 2014-02-03 21:40:50]

Committed patchset #6 manually as r248579 (presubmit successful).