Add input data parser for Mojo message validation tests.

Add input data parser for Mojo message validation tests.

BUG=None
TEST=None
R=darin@chromium.org

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

[yzshen1, 2014-06-06 21:56:04]

Hi, Darin.

Would you please take a look? Thanks!

[darin (slow to review), 2014-06-08 03:44:33]

LGTM

https://codereview.chromium.org/318123003/diff/1/mojo/public/cpp/bindings/tes...
File mojo/public/cpp/bindings/tests/validation_test_input_parser.cc (right):

https://codereview.chromium.org/318123003/diff/1/mojo/public/cpp/bindings/tes...
mojo/public/cpp/bindings/tests/validation_test_input_parser.cc:348: return
false;
nit: indentation

https://codereview.chromium.org/318123003/diff/1/mojo/public/cpp/bindings/tes...
File mojo/public/cpp/bindings/tests/validation_unittest.cc (right):

https://codereview.chromium.org/318123003/diff/1/mojo/public/cpp/bindings/tes...
mojo/public/cpp/bindings/tests/validation_unittest.cc:44: } else {
nit: no need for else after return

[yzshen1, 2014-06-08 06:59:13]

Thanks Darin!

https://codereview.chromium.org/318123003/diff/1/mojo/public/cpp/bindings/tes...
File mojo/public/cpp/bindings/tests/validation_test_input_parser.cc (right):

https://codereview.chromium.org/318123003/diff/1/mojo/public/cpp/bindings/tes...
mojo/public/cpp/bindings/tests/validation_test_input_parser.cc:348: return
false;
On 2014/06/08 03:44:33, darin wrote:
> nit: indentation

Done.

https://codereview.chromium.org/318123003/diff/1/mojo/public/cpp/bindings/tes...
File mojo/public/cpp/bindings/tests/validation_unittest.cc (right):

https://codereview.chromium.org/318123003/diff/1/mojo/public/cpp/bindings/tes...
mojo/public/cpp/bindings/tests/validation_unittest.cc:44: } else {
On 2014/06/08 03:44:33, darin wrote:
> nit: no need for else after return

Done.

[yzshen1, 2014-06-08 08:27:36]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2014-06-08 08:28:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yzshen@chromium.org/318123003/20001

[commit-bot: I haz the power, 2014-06-08 12:03:41]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[commit-bot: I haz the power, 2014-06-08 13:45:08]

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

[commit-bot: I haz the power, 2014-06-08 13:45:08]

Try jobs failed on following builders:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[yzshen1, 2014-06-09 06:38:54]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2014-06-09 06:39:49]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yzshen@chromium.org/318123003/40001

[commit-bot: I haz the power, 2014-06-09 08:29:00]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/8...)
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)

[commit-bot: I haz the power, 2014-06-09 08:53:47]

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

[commit-bot: I haz the power, 2014-06-09 08:53:48]

Try jobs failed on following builders:
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)

[yzshen1, 2014-06-09 17:30:52]

Committed patchset #5 manually as r275831 (presubmit successful).