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

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

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

R=dmazzoni@chromium.org
BUG=none

Committed: https://crrev.com/6daa58f92bff772a4476f761ef4c640bf90a2605
Cr-Commit-Position: refs/heads/master@{#316538}

[Dan Beam, 2015-02-14 02:47:27]

split off these changes because there were cvox-related try job failures.  let's
see if they happen again...

[dmazzoni, 2015-02-14 07:08:04]

dmazzoni@chromium.org changed reviewers:
+ dtseng@chromium.org, plundblad@chromium.org

[dmazzoni, 2015-02-14 07:08:05]

Fine with me, but Peter & David are the primary owners so I'll make sure it's
okay with them too.

[Peter Lundblad, 2015-02-16 16:34:48]

lgtm

Hi,

Nice!  Would you mind running this over our .extjs and .unitjs test files as
well? We don't (currently) compile these and there could very well be jsdoc
errors in those files. Still it'd be nice for consistency.  Sending lgtm in case
you'd rather do that in a separate cl.

[Dan Beam, 2015-02-17 06:45:33]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2015-02-17 06:45:58]

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

[commit-bot: I haz the power, 2015-02-17 06:49:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-17 06:51:08]

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

[Dan Beam, 2015-02-17 07:00:34]

On 2015/02/16 16:34:48, Peter Lundblad wrote:
> lgtm
> 
> Hi,
> 
> Nice!  Would you mind running this over our .extjs and .unitjs test files as
> well? We don't (currently) compile these and there could very well be jsdoc
> errors in those files. Still it'd be nice for consistency.  Sending lgtm in
case
> you'd rather do that in a separate cl.

will run on .extjs and .unitjs soon.  any reason why these aren't .js?

[Peter Lundblad, 2015-02-17 10:12:26]

dbeam@chromium.org writes:
> On 2015/02/16 16:34:48, Peter Lundblad wrote:
> > lgtm
> 
> > Hi,
> 
> > Nice!  Would you mind running this over our .extjs and .unitjs test files  
> > as
> > well? We don't (currently) compile these and there could very well be  
> > jsdoc
> > errors in those files. Still it'd be nice for consistency.  Sending lgtm  
> > in
> case
> > you'd rather do that in a separate cl.
> 
> will run on .extjs and .unitjs soon.  any reason why these aren't .js?

Yes, they are translated to C++, similar to the webui and js unit tests and
the gyp rules are selected based on the extension.

//Peter

> https://codereview.chromium.org/924083004/

-- 

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.