Simplify HTMLPlugInElement::PluginWrapper to not use SharedPersistent.

Simplify HTMLPlugInElement::PluginWrapper to not use SharedPersistent.

This handle is always uniquely owned, anyhow. And the indirection through
ScriptController adds very little value.

BUG=None

Review-Url: https://codereview.chromium.org/2855753002
Cr-Commit-Position: refs/heads/master@{#468625}
Committed: https://chromium.googlesource.com/chromium/src/+/8569024f3f1ef296d6614c9482b9de6ed14b3ced

[jbroman, 2017-05-01 21:32:45]

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

[commit-bot: I haz the power, 2017-05-01 21:33:41]

Dry run: CQ is trying da patch.

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

[jbroman, 2017-05-01 21:36:19]

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

[commit-bot: I haz the power, 2017-05-01 21:36:34]

Dry run: CQ is trying da patch.

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

[jbroman, 2017-05-01 23:24:16]

jbroman@chromium.org changed reviewers:
+ haraken@chromium.org

[jbroman, 2017-05-01 23:24:17]

The use of SharedPersistent (which is ref-counted) here is overkill. Simplify by
just using v8::Local/Global as appropriate.

(There are other ways this code might be improved, like looking at cross-world
usage and considering wrapper tracing, but such changes are not included in this
CL.)

[commit-bot: I haz the power, 2017-05-01 23:35:07]

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

[commit-bot: I haz the power, 2017-05-01 23:35:08]

Dry run: This issue passed the CQ dry run.

[haraken, 2017-05-02 04:45:54]

LGTM

[jbroman, 2017-05-02 13:58:03]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2017-05-02 13:58:17]

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-05-02 14:01:59]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493733483772730,
"parent_rev": "5bd9e2fd296b774e34d04cddfb09893525a79c04", "commit_rev":
"8569024f3f1ef296d6614c9482b9de6ed14b3ced"}

[commit-bot: I haz the power, 2017-05-02 14:02:05]

Description was changed from

==========
Simplify HTMLPlugInElement::PluginWrapper to not use SharedPersistent.

This handle is always uniquely owned, anyhow. And the indirection through
ScriptController adds very little value.

BUG=None
==========

to

==========
Simplify HTMLPlugInElement::PluginWrapper to not use SharedPersistent.

This handle is always uniquely owned, anyhow. And the indirection through
ScriptController adds very little value.

BUG=None

Review-Url: https://codereview.chromium.org/2855753002
Cr-Commit-Position: refs/heads/master@{#468625}
Committed:
https://chromium.googlesource.com/chromium/src/+/8569024f3f1ef296d6614c9482b9...
==========

[commit-bot: I haz the power, 2017-05-02 14:02:06]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/8569024f3f1ef296d6614c9482b9...