Remove T:: from ActiveScriptWrappable<T>::DispatchHasPendingActivity()

Remove T:: from ActiveScriptWrappable<T>::DispatchHasPendingActivity()

Previously, for a class T : public ActiveScriptWrappable<T>,
HasPendingActivity() cannot be overridden by the subclasses of T
because DispatchHasPendingActivity() calls T::HasPendingActivity().

This CL enables the override by calling HasPendingActivity() instead,
as preparation for https://codereview.chromium.org/2905233002/.

BUG=383741, 327574, 726091, 726414
Review-Url: https://codereview.chromium.org/2907173002
Cr-Commit-Position: refs/heads/master@{#475480}
Committed: https://chromium.googlesource.com/chromium/src/+/44f1ff3ef09322690900c00580c90b841a52d73a

[hiroshige, 2017-05-29 15:19:30]

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

[commit-bot: I haz the power, 2017-05-29 15:19:45]

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-05-29 17:15:28]

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

[commit-bot: I haz the power, 2017-05-29 17:15:29]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2017-05-30 08:49:33]

Description was changed from

==========
Remove T:: from DispatchHasPendingActivity()

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

to

==========
Remove T:: from ActiveScriptWrappable<T>::DispatchHasPendingActivity()

Previously, for a class T : public ActiveScriptWrappable<T>,
HasPendingActivity() cannot be overridden by the subclasses of T
because DispatchHasPendingActivity() calls T::HasPendingActivity().

This CL enables the override by calling HasPendingActivity() instead,
as preparation for https://codereview.chromium.org/2905233002/.

BUG=383741, 327574, 726091, 726414
==========

[hiroshige, 2017-05-30 08:49:33]

hiroshige@chromium.org changed reviewers:
+ haraken@chromium.org, kouhei@chromium.org

[hiroshige, 2017-05-30 08:51:38]

PTAL.

[hiroshige, 2017-05-30 08:52:52]

https://codereview.chromium.org/2907173002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/bindings/ActiveScriptWrappable.h
(right):

https://codereview.chromium.org/2907173002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/bindings/ActiveScriptWrappable.h:52:
(static_cast<T*>(object)->T::GetExecutionContext)()
Not blocking https://codereview.chromium.org/2905233002/ but also
IsContextDestroyed() has T::.

[haraken, 2017-05-30 08:54:10]

LGTM

[kouhei (in TOK), 2017-05-30 08:56:03]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2017-05-30 08:56:04]

lgtm

[commit-bot: I haz the power, 2017-05-30 08:56:24]

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

[commit-bot: I haz the power, 2017-05-30 08:56:25]

This CL has an open dependency (Issue 2912553002 Patch 1). Please resolve the
dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[hiroshige, 2017-05-30 09:34:38]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2017-05-30 09:34:52]

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-30 09:38:25]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496136878526320, "parent_rev":
"9c4d96f6224765185e9320832cb5ce5ab21ad748", "commit_rev":
"44f1ff3ef09322690900c00580c90b841a52d73a"}

[commit-bot: I haz the power, 2017-05-30 09:38:32]

Description was changed from

==========
Remove T:: from ActiveScriptWrappable<T>::DispatchHasPendingActivity()

Previously, for a class T : public ActiveScriptWrappable<T>,
HasPendingActivity() cannot be overridden by the subclasses of T
because DispatchHasPendingActivity() calls T::HasPendingActivity().

This CL enables the override by calling HasPendingActivity() instead,
as preparation for https://codereview.chromium.org/2905233002/.

BUG=383741, 327574, 726091, 726414
==========

to

==========
Remove T:: from ActiveScriptWrappable<T>::DispatchHasPendingActivity()

Previously, for a class T : public ActiveScriptWrappable<T>,
HasPendingActivity() cannot be overridden by the subclasses of T
because DispatchHasPendingActivity() calls T::HasPendingActivity().

This CL enables the override by calling HasPendingActivity() instead,
as preparation for https://codereview.chromium.org/2905233002/.

BUG=383741, 327574, 726091, 726414

Review-Url: https://codereview.chromium.org/2907173002
Cr-Commit-Position: refs/heads/master@{#475480}
Committed:
https://chromium.googlesource.com/chromium/src/+/44f1ff3ef09322690900c00580c9...
==========

[commit-bot: I haz the power, 2017-05-30 09:38:35]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/44f1ff3ef09322690900c00580c9...