Shorten Closure template notation from Array.<*> to Array<*>.

Shorten Closure template notation from Array.<*> to Array<*>.

These are equivalent syntaxes, the latter is just newer and shorter
and more closely resembles C++/Java.

R=brettw@chromium.org
BUG=none

Committed: https://chromium.googlesource.com/chromium/src/+/b9749f4141f5ceef300b049b4d55b06089798fee

[michaelpg, 2015-02-12 23:40:41]

sorry, drive-by question... how can you change the annotation syntax if we'll
want to compile/lint these w/ closure?

[dmazzoni, 2015-02-12 23:43:49]

dmazzoni@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[dmazzoni, 2015-02-12 23:43:51]

Is the Closure Compiler documentation out of date?

https://developers.google.com/closure/compiler/docs/js-for-compiler

It still suggests Array.<*>.

[Dan Beam, 2015-02-13 00:15:52]

On 2015/02/12 23:40:41, michaelpg wrote:
> sorry, drive-by question... how can you change the annotation syntax if we'll
> want to compile/lint these w/ closure?

updated the CL description to further explain that it's an equivalent syntax.

this is similar to

  std::vector<std::vector<int> >

changing to

  std::vector<std::vector<int>>

not functionally different; just prettier and more sane.


On 2015/02/12 23:43:51, dmazzoni wrote:
> Is the Closure Compiler documentation out of date?
> 
> https://developers.google.com/closure/compiler/docs/js-for-compiler
> 
> It still suggests Array.<*>.

For back compatibility with old compilers, I'd assume.  This is a new-ish
feature:
https://github.com/google/closure-compiler/wiki/Releases#october-23-2014-v201...

[Dan Beam, 2015-02-13 02:07:27]

dbeam@chromium.org changed reviewers:
- dmazzoni@chromium.org

[Dan Beam, 2015-02-13 19:22:09]

ping brettw@

[brettw, 2015-02-13 20:29:45]

On 2015/02/13 19:22:09, Dan Beam wrote:
> ping brettw@

I'm happy to owners stamp this but I don't know anything about JS so I'm not
able to say if this is correct or not. Can somebody knowledgeable review first?

[Evan Stade, 2015-02-13 20:48:53]

On 2015/02/13 20:29:45, brettw wrote:
> On 2015/02/13 19:22:09, Dan Beam wrote:
> > ping brettw@
> 
> I'm happy to owners stamp this but I don't know anything about JS so I'm not
> able to say if this is correct or not. Can somebody knowledgeable review
first?

looks legit:
https://github.com/google/closure-compiler/wiki/Releases#october-23-2014-v201...

lgtm

[brettw, 2015-02-13 20:53:49]

rs lgtm

[Dan Beam, 2015-02-13 22:13:59]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2015-02-13 22:14:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/917093003/20001

[commit-bot: I haz the power, 2015-02-13 22:24:16]

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

[commit-bot: I haz the power, 2015-02-13 22:24:20]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Dan Beam, 2015-02-13 22:31:59]

New patchsets have been uploaded after l-g-t-m from
estade@chromium.org,brettw@chromium.org

[Dan Beam, 2015-02-13 22:32:16]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2015-02-13 22:33:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/917093003/40001

[commit-bot: I haz the power, 2015-02-13 23:01:08]

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

[commit-bot: I haz the power, 2015-02-13 23:01:12]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Dan Beam, 2015-02-14 02:38:29]

Patchset #6 (id:100001) has been deleted

[commit-bot: I haz the power, 2015-02-14 02:44:11]

Patchset 6 (id:??) landed as
https://crrev.com/b9749f4141f5ceef300b049b4d55b06089798fee
Cr-Commit-Position: refs/heads/master@{#316365}

[Dan Beam, 2015-02-14 02:44:26]

Committed patchset #6 (id:120001) manually as
b9749f4141f5ceef300b049b4d55b06089798fee (presubmit successful).

[Tyler Breisacher (Chromium), 2015-02-15 21:37:23]

On 2015/02/12 23:43:51, dmazzoni wrote:
> Is the Closure Compiler documentation out of date?
> 
> https://developers.google.com/closure/compiler/docs/js-for-compiler
> 
> It still suggests Array.<*>.

I updated the docs by searching for '.<' in the HTML, but I missed several
instances because they're '.&lt;'

Thanks for the heads-up and please let us know if you find any other out-of-date
docs!