InlinedVisitor: Migrate templated classes in bindings/ manually to use inlined tracing

InlinedVisitor: Migrate templated classes in bindings/ manually to use inlined tracing

Updates trace methods as of https://groups.google.com/a/chromium.org/d/msg/blink-dev/oi4Snt9jNx8/XcES55c7QaQJ

Note to reviewers:
This CL is created manually, as DEFINE_TRACE macros doesn't support template methods.

R=oilpan-reviews
BUG=455524
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190719

[kouhei (in TOK), 2015-02-20 02:14:09]

kouhei@chromium.org changed reviewers:
+ yhirano@chromium.org

[tkent, 2015-02-20 03:22:55]

tkent@chromium.org changed reviewers:
+ tkent@chromium.org

[tkent, 2015-02-20 03:22:55]

The CQ bit was checked by tkent@chromium.org

[tkent, 2015-02-20 03:22:56]

lgtm

[commit-bot: I haz the power, 2015-02-20 03:23:39]

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

[yhirano, 2015-02-20 03:27:23]

Do you need updating the following files?
 - bindings/templates/callback_interface.cpp
 - bindings/templates/dictionary_impl.cpp
 - bindings/templates/dictionary_impl.h
 - bindings/templates/union.cpp
 - bindings/templates/union.h

Or you intentionally exclude these files because they can be automatically
updated? If so, it's OK.

[commit-bot: I haz the power, 2015-02-20 03:36:29]

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

[commit-bot: I haz the power, 2015-02-20 03:36:31]

Try jobs failed on following builders:
  mac_blink_compile_dbg on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)

[kouhei (in TOK), 2015-02-20 03:46:42]

> intentionally exclude these files because they can be automatically
> updated? If so, it's OK.

Yes. I'll run script on it on separate CL.

[yhirano, 2015-02-20 04:22:14]

On 2015/02/20 03:46:42, kouhei wrote:
> > intentionally exclude these files because they can be automatically
> > updated? If so, it's OK.
> 
> Yes. I'll run script on it on separate CL.

I see.
By the way traceImpl is not declared if !ENABLE(INLINE_TRACE).

[kouhei (in TOK), 2015-02-24 01:29:26]

On 2015/02/20 04:22:14, yhirano wrote:
> On 2015/02/20 03:46:42, kouhei wrote:
> > > intentionally exclude these files because they can be automatically
> > > updated? If so, it's OK.
> > 
> > Yes. I'll run script on it on separate CL.
> 
> I see.
> By the way traceImpl is not declared if !ENABLE(INLINE_TRACE).

Thanks. Fixed.

[yhirano, 2015-02-24 04:30:19]

lgtm

https://codereview.chromium.org/940293002/diff/20001/Source/bindings/core/v8/...
File Source/bindings/core/v8/ScriptPromiseProperty.h (right):

https://codereview.chromium.org/940293002/diff/20001/Source/bindings/core/v8/...
Source/bindings/core/v8/ScriptPromiseProperty.h:168: 
An unnecessary empty line

[kouhei (in TOK), 2015-02-24 04:35:16]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2015-02-24 04:35:16]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
yhirano@chromium.org
Link to the patchset: https://codereview.chromium.org/940293002/#ps40001 (title:
"remove line")

[commit-bot: I haz the power, 2015-02-24 04:35:55]

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

[kouhei (in TOK), 2015-02-24 04:37:15]

https://codereview.chromium.org/940293002/diff/20001/Source/bindings/core/v8/...
File Source/bindings/core/v8/ScriptPromiseProperty.h (right):

https://codereview.chromium.org/940293002/diff/20001/Source/bindings/core/v8/...
Source/bindings/core/v8/ScriptPromiseProperty.h:168: 
On 2015/02/24 04:30:19, yhirano wrote:
> An unnecessary empty line

Done.

[commit-bot: I haz the power, 2015-02-24 07:16:43]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=190719

[haraken, 2015-02-24 08:43:41]

LGTM