Port Mojo's sample_service.cc to JavaScript

Port Mojo's sample_service.cc to JavaScript

This CL completes the manual implementation of sample_service.js and tests it
using sample_test.js, which exactly mirrors the testing of sample_service.cc by
sample_service.cc.

(I've also moved sample_service.js into mojo/public/bindings/sample/mojom to
better mirror the C++ implementation.)

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

[abarth-chromium, 2013-11-18 23:30:43]

Please take a look.  Thanks!

https://codereview.chromium.org/68323004/diff/1/mojo/public/bindings/js/codec.js
File mojo/public/bindings/js/codec.js (left):

https://codereview.chromium.org/68323004/diff/1/mojo/public/bindings/js/codec...
mojo/public/bindings/js/codec.js:394: exports.load32 = load32;
We don't need to export load32 anymore.  We used to export that for some
low-level tests that are now subsumed by higher level testing.

https://codereview.chromium.org/68323004/diff/1/mojo/public/bindings/js/codec.js
File mojo/public/bindings/js/codec.js (right):

https://codereview.chromium.org/68323004/diff/1/mojo/public/bindings/js/codec...
mojo/public/bindings/js/codec.js:56: return size + (kAlignment - (size %
kAlignment)) % kAlignment;
The last % kAlignment is needed to handle the size == 8 case.

https://codereview.chromium.org/68323004/diff/1/mojo/public/bindings/sample/m...
File mojo/public/bindings/sample/mojom/sample_service.js (right):

https://codereview.chromium.org/68323004/diff/1/mojo/public/bindings/sample/m...
mojo/public/bindings/sample/mojom/sample_service.js:11: // Bar
----------------------------------------------------------------------
git cl upload didn't detect the move, but this code is mostly just moved from
the old location.

https://codereview.chromium.org/68323004/diff/1/mojo/public/bindings/sample/m...
mojo/public/bindings/sample/mojom/sample_service.js:96: // Service
------------------------------------------------------------------
The new code is from here down.

https://codereview.chromium.org/68323004/diff/1/mojo/public/bindings/sample/s...
File mojo/public/bindings/sample/sample_test.js (right):

https://codereview.chromium.org/68323004/diff/1/mojo/public/bindings/sample/s...
mojo/public/bindings/sample/sample_test.js:12: ], function(gtest, codec, sample)
{
This code is a direct transliteration of sample_test.cc without the extra print
statements.

[Aaron Boodman, 2013-11-19 01:09:55]

lgtm

[commit-bot: I haz the power, 2013-11-19 01:37:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/abarth@chromium.org/68323004/1

[commit-bot: I haz the power, 2013-11-19 04:33:17]

Change committed as 235917