Remove deprecated ExceptionState constructors.

Remove deprecated ExceptionState constructors.

BUG=673090
Committed: https://crrev.com/8ca234a6e47d326f9b4513fe93b86ff7afd162e0
Cr-Commit-Position: refs/heads/master@{#437773}

[dcheng, 2016-12-10 06:59:32]

Description was changed from

==========
Remove deprecated ExceptionState constructors.

BUG=
==========

to

==========
Remove deprecated ExceptionState constructors.

BUG=673090
==========

[dcheng, 2016-12-10 06:59:49]

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

[commit-bot: I haz the power, 2016-12-10 07:00:04]

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

[dcheng, 2016-12-10 07:42:12]

dcheng@chromium.org changed reviewers:
+ haraken@chromium.org, yukishiino@chromium.org

[dcheng, 2016-12-10 07:42:13]

https://codereview.chromium.org/2566823002/diff/1/third_party/WebKit/Source/b...
File third_party/WebKit/Source/bindings/templates/interface.cpp.tmpl (right):

https://codereview.chromium.org/2566823002/diff/1/third_party/WebKit/Source/b...
third_party/WebKit/Source/bindings/templates/interface.cpp.tmpl:504:
ExceptionState exceptionState(info.GetIsolate(), ExceptionState::SetterContext,
"{{interface_name}}", *attributeName);
I thought about adding a TODO: with the ExceptionState changes I'm planning (to
make property name a StringView), we could defer this conversion until we
actually throw the exception. However, this isn't used in many situations... and
in all the situations we use it in, the input string should be ASCII. So
actually changing this doesn't save us much, I think.

[commit-bot: I haz the power, 2016-12-10 09:50:32]

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

[commit-bot: I haz the power, 2016-12-10 09:50:33]

Dry run: This issue passed the CQ dry run.

[haraken, 2016-12-10 12:56:09]

LGTM

[dcheng, 2016-12-10 18:14:34]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2016-12-10 18:14:46]

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, 2016-12-10 18:18:42]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481393674611570, "parent_rev":
"d9951ac8c08290b8d1a5b0eb8a160f85eb65dbde", "commit_rev":
"82c203a46e81039323ebfe5b4e828b2f4517c986"}

[commit-bot: I haz the power, 2016-12-10 18:19:12]

Description was changed from

==========
Remove deprecated ExceptionState constructors.

BUG=673090
==========

to

==========
Remove deprecated ExceptionState constructors.

BUG=673090

Review-Url: https://codereview.chromium.org/2566823002
==========

[commit-bot: I haz the power, 2016-12-10 18:19:13]

Committed patchset #1 (id:1)

[Yuki, 2016-12-12 03:45:04]

LGTM.  Thanks for doing this.

[commit-bot: I haz the power, 2016-12-12 15:07:09]

Description was changed from

==========
Remove deprecated ExceptionState constructors.

BUG=673090

Review-Url: https://codereview.chromium.org/2566823002
==========

to

==========
Remove deprecated ExceptionState constructors.

BUG=673090

Committed: https://crrev.com/8ca234a6e47d326f9b4513fe93b86ff7afd162e0
Cr-Commit-Position: refs/heads/master@{#437773}
==========

[commit-bot: I haz the power, 2016-12-12 15:07:10]

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