Rename ScriptWrappable::newLocalWrapper to ScriptWrappable::mainWorldWrapper

Rename ScriptWrappable::newLocalWrapper to ScriptWrappable::mainWorldWrapper

To clarify that it returns a main world's wrapper.
To avoid the method from being used randomly, this CL makes the method private.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel

Committed: https://crrev.com/2658bb52232e9d1fcb4b45b2846f4b69e9e44800
Cr-Commit-Position: refs/heads/master@{#396796}

[haraken, 2016-05-31 04:37:25]

Description was changed from

==========
Rename ScriptWrappable::newLocalWrapper to ScriptWrappable::mainWorldWrapper

To clarify that it returns a main world's wrapper.
To avoid the method from being used randomly, this CL makes the method private.

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

to

==========
Rename ScriptWrappable::newLocalWrapper to ScriptWrappable::mainWorldWrapper

To clarify that it returns a main world's wrapper.
To avoid the method from being used randomly, this CL makes the method private.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[haraken, 2016-05-31 04:40:21]

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

[haraken, 2016-05-31 04:40:22]

PTAL

(This is a change requested by kbr offline.)

[Yuki, 2016-05-31 04:43:11]

lgtm

https://codereview.chromium.org/2021203002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h (right):

https://codereview.chromium.org/2021203002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h:167: friend class
DOMDataStore;
Maybe, better to comment why these classes are friend.

[haraken, 2016-05-31 04:44:33]

https://codereview.chromium.org/2021203002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h (right):

https://codereview.chromium.org/2021203002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h:167: friend class
DOMDataStore;
On 2016/05/31 04:43:11, Yuki wrote:
> Maybe, better to comment why these classes are friend.

Done.

[haraken, 2016-05-31 04:44:54]

The CQ bit was checked by haraken@chromium.org

[haraken, 2016-05-31 04:44:54]

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

[commit-bot: I haz the power, 2016-05-31 04:45:03]

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

[commit-bot: I haz the power, 2016-05-31 07:02:42]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-31 07:03:51]

Description was changed from

==========
Rename ScriptWrappable::newLocalWrapper to ScriptWrappable::mainWorldWrapper

To clarify that it returns a main world's wrapper.
To avoid the method from being used randomly, this CL makes the method private.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Rename ScriptWrappable::newLocalWrapper to ScriptWrappable::mainWorldWrapper

To clarify that it returns a main world's wrapper.
To avoid the method from being used randomly, this CL makes the method private.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel

Committed: https://crrev.com/2658bb52232e9d1fcb4b45b2846f4b69e9e44800
Cr-Commit-Position: refs/heads/master@{#396796}
==========

[commit-bot: I haz the power, 2016-05-31 07:03:52]

Patchset 3 (id:??) landed as
https://crrev.com/2658bb52232e9d1fcb4b45b2846f4b69e9e44800
Cr-Commit-Position: refs/heads/master@{#396796}