Rename cancelValuesAndHoldAtTime to cancelAndHoldAtTime.

Rename cancelValuesAndHoldAtTime to cancelAndHoldAtTime.

The actual automation name is cancelAndHoldAtTime, not
cancelValuesAndHoldAtTime:
https://webaudio.github.io/web-audio-api/#widl-AudioParam-cancelAndHoldAtTime-AudioParam-double-cancelTime

Make it so.

BUG=686242
TEST=AudioParam/audioparam-cancel-and-hold.html

Review-Url: https://codereview.chromium.org/2655073009
Cr-Commit-Position: refs/heads/master@{#447281}
Committed: https://chromium.googlesource.com/chromium/src/+/f5e8986bdcd359c7ca82d83907c81642dd01f55e

[Raymond Toy, 2017-01-27 22:09:08]

rtoy@chromium.org changed reviewers:
+ hongchan@chromium.org, tkent@chromium.org

[Raymond Toy, 2017-01-27 22:09:09]

PTAL.

We mistakenly implemented the wrong name.

I didn't write an intent for this.  Do I need one?  It's clearly a typo that no
one caught. (The name changed quite a few times between when I first started the
work and when it finally got spec'ed.)

[Raymond Toy, 2017-01-27 22:11:54]

The CQ bit was checked by rtoy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-27 22:12:33]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-27 23:36:51]

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

[commit-bot: I haz the power, 2017-01-27 23:36:52]

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

[tkent, 2017-01-30 00:08:29]

On 2017/01/27 at 22:09:09, rtoy wrote:
> PTAL.
> 
> We mistakenly implemented the wrong name.
> 
> I didn't write an intent for this.  Do I need one?  It's clearly a typo that
no one caught. (The name changed quite a few times between when I first started
the work and when it finally got spec'ed.)

It isn't shipped in the stable channel yet, right?  If it's right, let's rename
it without an intent, and merge it to a branch if necessary.

[Raymond Toy, 2017-01-30 16:36:07]

On 2017/01/30 00:08:29, tkent wrote:
> On 2017/01/27 at 22:09:09, rtoy wrote:
> > PTAL.
> > 
> > We mistakenly implemented the wrong name.
> > 
> > I didn't write an intent for this.  Do I need one?  It's clearly a typo that
> no one caught. (The name changed quite a few times between when I first
started
> the work and when it finally got spec'ed.)
> 
> It isn't shipped in the stable channel yet, right?  If it's right, let's
rename
> it without an intent, and merge it to a branch if necessary.

The original CL landed a couple of weeks ago, just before the M57 branch.  My
intent was to merge this to M57.

[hongchan, 2017-01-30 18:02:37]

lgtm

[tkent, 2017-01-30 23:00:48]

lgtm

[Raymond Toy, 2017-01-30 23:03:59]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-01-30 23:04:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-31 00:53:56]

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

[commit-bot: I haz the power, 2017-01-31 00:53:57]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Raymond Toy, 2017-01-31 16:38:13]

The CQ bit was checked by rtoy@chromium.org

[Raymond Toy, 2017-01-31 16:38:14]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
hongchan@chromium.org
Link to the patchset: https://codereview.chromium.org/2655073009/#ps40001
(title: "Rebase expected results")

[commit-bot: I haz the power, 2017-01-31 16:38:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-31 18:47:17]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485880693850790,
"parent_rev": "0844eead6025c71d51db860f4a0290ea32f7aea9", "commit_rev":
"f5e8986bdcd359c7ca82d83907c81642dd01f55e"}

[commit-bot: I haz the power, 2017-01-31 18:48:06]

Description was changed from

==========
Rename cancelValuesAndHoldAtTime to cancelAndHoldAtTime.

The actual automation name is cancelAndHoldAtTime, not
cancelValuesAndHoldAtTime:
https://webaudio.github.io/web-audio-api/#widl-AudioParam-cancelAndHoldAtTime...

Make it so.

BUG=686242
TEST=AudioParam/audioparam-cancel-and-hold.html
==========

to

==========
Rename cancelValuesAndHoldAtTime to cancelAndHoldAtTime.

The actual automation name is cancelAndHoldAtTime, not
cancelValuesAndHoldAtTime:
https://webaudio.github.io/web-audio-api/#widl-AudioParam-cancelAndHoldAtTime...

Make it so.

BUG=686242
TEST=AudioParam/audioparam-cancel-and-hold.html

Review-Url: https://codereview.chromium.org/2655073009
Cr-Commit-Position: refs/heads/master@{#447281}
Committed:
https://chromium.googlesource.com/chromium/src/+/f5e8986bdcd359c7ca82d83907c8...
==========

[commit-bot: I haz the power, 2017-01-31 18:48:07]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/f5e8986bdcd359c7ca82d83907c8...