Add module annotation to specify the name of the java class for constants.

Add module annotation to specify the name of the java class for constants.

R=rmcilroy@chromium.org, viettrungluu@chromium.org

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

[rmcilroy, 2014-05-28 17:03:30]

nice, lgtm - thanks.

https://codereview.chromium.org/300223008/diff/1/mojo/public/interfaces/bindi...
File mojo/public/interfaces/bindings/tests/sample_interfaces.mojom (right):

https://codereview.chromium.org/300223008/diff/1/mojo/public/interfaces/bindi...
mojo/public/interfaces/bindings/tests/sample_interfaces.mojom:10: const uint64
kLong = (879+2)*5;
nit - spaces between binary operators.  Could we also have a String const here
too in order to test that?

[viettrungluu, 2014-05-28 19:10:26]

lgtm

https://codereview.chromium.org/300223008/diff/1/mojo/public/interfaces/bindi...
File mojo/public/interfaces/bindings/tests/sample_interfaces.mojom (right):

https://codereview.chromium.org/300223008/diff/1/mojo/public/interfaces/bindi...
mojo/public/interfaces/bindings/tests/sample_interfaces.mojom:10: const uint64
kLong = (879+2)*5;
On 2014/05/28 17:03:30, rmcilroy wrote:
> nit - spaces between binary operators.  Could we also have a String const here
> too in order to test that?

Note that I'm going to remove all expressions shortly. And probably string
constants too.

[qsr, 2014-06-03 08:58:59]

The CQ bit was checked by qsr@chromium.org

[commit-bot: I haz the power, 2014-06-03 09:00:56]

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

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

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...)

[qsr, 2014-06-03 14:54:23]

Committed patchset #2 manually as r274526 (presubmit successful).