Refactor ScriptWrappableVisitor.markWrapper(PersistentBase) to be instance method

Refactor ScriptWrappableVisitor.markWrapper(PersistentBase) to be instance method

There is no reason why markWrapper should be static method, and making it
instance method makes it easier to use other visitors (for example verifying
visitor for write barrier checking comes to my mind :)

There is a little fight in me whether to call the method markWrapper or
traceWrappers. I went with traceWrappers in this cl for consistentcy in all
traceWrappers methods. In either case, the same name should be used for
ScopedPersistents too.

LOG=no
BUG=468240
Committed: https://crrev.com/03b8da6059cd836993aaea1039a2be19bdcead4f
Cr-Commit-Position: refs/heads/master@{#400974}

[Marcel Hlopko, 2016-06-20 16:39:25]

hlopko@chromium.org changed reviewers:
+ haraken@chromium.org, hpayer@chromium.org, jochen@chromium.org

[Marcel Hlopko, 2016-06-20 16:40:06]

Ptal.

[haraken, 2016-06-20 23:35:09]

LGTM

[Marcel Hlopko, 2016-06-21 07:58:37]

The CQ bit was checked by hlopko@chromium.org

[commit-bot: I haz the power, 2016-06-21 07:58:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2086553002/1

[commit-bot: I haz the power, 2016-06-21 08:00:25]

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

[commit-bot: I haz the power, 2016-06-21 08:00:27]

Try jobs failed on following builders:
  ios-device-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)
  ios-simulator on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)

[Marcel Hlopko, 2016-06-21 08:20:42]

The CQ bit was checked by hlopko@chromium.org

[Marcel Hlopko, 2016-06-21 08:20:43]

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

[commit-bot: I haz the power, 2016-06-21 08:21:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2086553002/20001

[Marcel Hlopko, 2016-06-21 09:18:43]

The CQ bit was unchecked by hlopko@chromium.org

[Marcel Hlopko, 2016-06-21 11:46:29]

The CQ bit was checked by hlopko@chromium.org

[Marcel Hlopko, 2016-06-21 11:46:29]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2086553002/#ps80001
(title: "Whitespace fix")

[commit-bot: I haz the power, 2016-06-21 11:46:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2086553002/80001

[commit-bot: I haz the power, 2016-06-21 13:20:39]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-06-21 13:23:49]

Description was changed from

==========
Refactor ScriptWrappableVisitor.markWrapper(PersistentBase) to be instance
method

There is no reason why markWrapper should be static method, and making it
instance method makes it easier to use other visitors (for example verifying
visitor for write barrier checking comes to my mind :)

There is a little fight in me whether to call the method markWrapper or
traceWrappers. I went with traceWrappers in this cl for consistentcy in all
traceWrappers methods. In either case, the same name should be used for
ScopedPersistents too.

LOG=no
BUG=468240
==========

to

==========
Refactor ScriptWrappableVisitor.markWrapper(PersistentBase) to be instance
method

There is no reason why markWrapper should be static method, and making it
instance method makes it easier to use other visitors (for example verifying
visitor for write barrier checking comes to my mind :)

There is a little fight in me whether to call the method markWrapper or
traceWrappers. I went with traceWrappers in this cl for consistentcy in all
traceWrappers methods. In either case, the same name should be used for
ScopedPersistents too.

LOG=no
BUG=468240

Committed: https://crrev.com/03b8da6059cd836993aaea1039a2be19bdcead4f
Cr-Commit-Position: refs/heads/master@{#400974}
==========

[commit-bot: I haz the power, 2016-06-21 13:23:50]

Patchset 5 (id:??) landed as
https://crrev.com/03b8da6059cd836993aaea1039a2be19bdcead4f
Cr-Commit-Position: refs/heads/master@{#400974}