Port refcount-leaks to PPAPI

Port refcount-leaks to PPAPI

Trying to keep the spirit of the test, but refcount is not exposed to the plugin in PPAPI, so just focus on the instance count.

BUG=318978, 493212, 474535
Committed: https://crrev.com/ef6f7853d2fe2da55cf97718135d572a42aa841d
Cr-Commit-Position: refs/heads/master@{#383677}

[piman, 2016-03-29 03:04:44]

Patchset #1 (id:1) has been deleted

[piman, 2016-03-29 03:05:46]

Description was changed from

==========
Port refcount-leaks to PPAPI

BUG=318978,493212,474535
==========

to

==========
Port refcount-leaks to PPAPI

Trying to keep the spirit of the test, but refcount is not exposed to the plugin
in PPAPI, so just focus on the instance count.

BUG=318978,493212,474535
==========

[piman, 2016-03-29 03:05:58]

piman@chromium.org changed reviewers:
+ dcheng@chromium.org

[piman, 2016-03-29 03:06:24]

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

[commit-bot: I haz the power, 2016-03-29 03:06:34]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1840963002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1840963002/20001

[dcheng, 2016-03-29 03:15:24]

lgtm

https://codereview.chromium.org/1840963002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/plugins/refcount-leaks.html (right):

https://codereview.chromium.org/1840963002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/plugins/refcount-leaks.html:45: 
No test expectations here because we can't get to the refcount of the scriptable
object, right? Maybe add a comment to that effect?

[piman, 2016-03-29 05:09:29]

https://codereview.chromium.org/1840963002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/plugins/refcount-leaks.html (right):

https://codereview.chromium.org/1840963002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/plugins/refcount-leaks.html:45: 
On 2016/03/29 03:15:24, dcheng wrote:
> No test expectations here because we can't get to the refcount of the
scriptable
> object, right? Maybe add a comment to that effect?

Oh, yeah I should add a comment. This will refcount the object by passing it to
the plugin and again by the plugin calling noop, and we will verify in
finishTest that the refcounts are properly unwound, by verifying the plugin
object was properly deleted.

[piman, 2016-03-29 05:13:26]

The CQ bit was checked by piman@chromium.org

[piman, 2016-03-29 05:13:26]

The patchset sent to the CQ was uploaded after l-g-t-m from dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/1840963002/#ps40001
(title: "add comment")

[commit-bot: I haz the power, 2016-03-29 05:13:35]

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

[commit-bot: I haz the power, 2016-03-29 06:11:49]

Description was changed from

==========
Port refcount-leaks to PPAPI

Trying to keep the spirit of the test, but refcount is not exposed to the plugin
in PPAPI, so just focus on the instance count.

BUG=318978,493212,474535
==========

to

==========
Port refcount-leaks to PPAPI

Trying to keep the spirit of the test, but refcount is not exposed to the plugin
in PPAPI, so just focus on the instance count.

BUG=318978,493212,474535
==========

[commit-bot: I haz the power, 2016-03-29 06:11:50]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-03-29 06:13:01]

Description was changed from

==========
Port refcount-leaks to PPAPI

Trying to keep the spirit of the test, but refcount is not exposed to the plugin
in PPAPI, so just focus on the instance count.

BUG=318978,493212,474535
==========

to

==========
Port refcount-leaks to PPAPI

Trying to keep the spirit of the test, but refcount is not exposed to the plugin
in PPAPI, so just focus on the instance count.

BUG=318978,493212,474535

Committed: https://crrev.com/ef6f7853d2fe2da55cf97718135d572a42aa841d
Cr-Commit-Position: refs/heads/master@{#383677}
==========

[commit-bot: I haz the power, 2016-03-29 06:13:02]

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