Rename ActiveDOMObject to SuspendableObject

Rename ActiveDOMObject to SuspendableObject

This is a rename requested by https://codereview.chromium.org/2554813002/.

Previously ActiveDOMObject was used to implement hasPendingActivity() of DOM wrappers,
where the name "ActiveDOMObject" made sense. Then we moved hasPendingActivity() to
ActiveScriptWrappable. Now ActiveDOMObject is a class that just provides resume/suspend.
It would make more sense to rename it to SuspendableObject.

BUG=610176
TBR=jam@chromium.org

Committed: https://crrev.com/7b372233ed3d9859e1204aa3a9af7f5f9bb32a56
Cr-Commit-Position: refs/heads/master@{#437820}

[haraken, 2016-12-12 00:41:29]

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

[haraken, 2016-12-12 00:41:30]

PTAL

[dcheng, 2016-12-12 02:06:27]

lgtm

https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/dom/ExecutionContext.h (right):

https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/dom/ExecutionContext.h:150: // suspend
Nit: merge comments on line 150 and 151

https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp (right):

https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp:71: //    the
Nit: can we fix the wrapping while we're changing this comment =)

https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp:74: //      we
Here as well.

[haraken, 2016-12-12 02:08:58]

Description was changed from

==========
Rename ActiveDOMObject to SuspendableObject

This is a rename requested by https://codereview.chromium.org/2554813002/.

Previously ActiveDOMObject was used to implement hasPendingActivity() of DOM
wrappers,
where the name "ActiveDOMObject" made sense. Then we moved hasPendingActivity()
to
ActiveScriptWrappable. Now ActiveDOMObject is a class that just provides
resume/suspend.
It would make more sense to rename it to SuspendableObject.

BUG=610176
==========

to

==========
Rename ActiveDOMObject to SuspendableObject

This is a rename requested by https://codereview.chromium.org/2554813002/.

Previously ActiveDOMObject was used to implement hasPendingActivity() of DOM
wrappers,
where the name "ActiveDOMObject" made sense. Then we moved hasPendingActivity()
to
ActiveScriptWrappable. Now ActiveDOMObject is a class that just provides
resume/suspend.
It would make more sense to rename it to SuspendableObject.

BUG=610176
TBR=jam@chromium.org
==========

[haraken, 2016-12-12 02:08:58]

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

[haraken, 2016-12-12 02:09:31]

>
https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/c...
> File third_party/WebKit/Source/core/dom/ExecutionContext.h (right):
> 
>
https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/c...
> third_party/WebKit/Source/core/dom/ExecutionContext.h:150: // suspend
> Nit: merge comments on line 150 and 151
> 
>
https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/m...
> File third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp
(right):
> 
>
https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/m...
> third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp:71: //   
the
> Nit: can we fix the wrapping while we're changing this comment =)
> 
>
https://codereview.chromium.org/2567913002/diff/1/third_party/WebKit/Source/m...
> third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp:74: //     
we
> Here as well.

Done.

TBRing jam@ for the trivial changes in content/.

[haraken, 2016-12-12 02:09:39]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-12 02:09:54]

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-12 04:30:43]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481508579740530, "parent_rev":
"c3d5520f37c12d0368fff98c4c3a35eecd95f480", "commit_rev":
"2037ed3c228e0b1837eeaefcd739f79ab18c9491"}

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

Description was changed from

==========
Rename ActiveDOMObject to SuspendableObject

This is a rename requested by https://codereview.chromium.org/2554813002/.

Previously ActiveDOMObject was used to implement hasPendingActivity() of DOM
wrappers,
where the name "ActiveDOMObject" made sense. Then we moved hasPendingActivity()
to
ActiveScriptWrappable. Now ActiveDOMObject is a class that just provides
resume/suspend.
It would make more sense to rename it to SuspendableObject.

BUG=610176
TBR=jam@chromium.org
==========

to

==========
Rename ActiveDOMObject to SuspendableObject

This is a rename requested by https://codereview.chromium.org/2554813002/.

Previously ActiveDOMObject was used to implement hasPendingActivity() of DOM
wrappers,
where the name "ActiveDOMObject" made sense. Then we moved hasPendingActivity()
to
ActiveScriptWrappable. Now ActiveDOMObject is a class that just provides
resume/suspend.
It would make more sense to rename it to SuspendableObject.

BUG=610176
TBR=jam@chromium.org

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

[commit-bot: I haz the power, 2016-12-12 04:31:12]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Rename ActiveDOMObject to SuspendableObject

This is a rename requested by https://codereview.chromium.org/2554813002/.

Previously ActiveDOMObject was used to implement hasPendingActivity() of DOM
wrappers,
where the name "ActiveDOMObject" made sense. Then we moved hasPendingActivity()
to
ActiveScriptWrappable. Now ActiveDOMObject is a class that just provides
resume/suspend.
It would make more sense to rename it to SuspendableObject.

BUG=610176
TBR=jam@chromium.org

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

to

==========
Rename ActiveDOMObject to SuspendableObject

This is a rename requested by https://codereview.chromium.org/2554813002/.

Previously ActiveDOMObject was used to implement hasPendingActivity() of DOM
wrappers,
where the name "ActiveDOMObject" made sense. Then we moved hasPendingActivity()
to
ActiveScriptWrappable. Now ActiveDOMObject is a class that just provides
resume/suspend.
It would make more sense to rename it to SuspendableObject.

BUG=610176
TBR=jam@chromium.org

Committed: https://crrev.com/7b372233ed3d9859e1204aa3a9af7f5f9bb32a56
Cr-Commit-Position: refs/heads/master@{#437820}
==========

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

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