Move IsPointerConvertible<T, JSONValue> specialization out of TypeTraits.h

Move IsPointerConvertible<T, JSONValue> specialization out of TypeTraits.h

Preferable to have it next to the type's declaration.

R=tkent,haraken

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=191688

[sof, 2015-03-10 15:46:04]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, tkent@chromium.org

[sof, 2015-03-10 15:46:04]

please take a look.

Avoids having any specializations to consider when resolving
IsPointerConvertible<> / EnableIf<> in most cases, fwiw.

[haraken, 2015-03-10 15:57:13]

LGTM

[tkent, 2015-03-11 00:32:50]

lgtm if JSONValue.h is included in the code using IsPointerConvertible<T,
JSONValue>.

[sof, 2015-03-11 06:24:44]

On 2015/03/11 00:32:50, tkent wrote:
> lgtm if JSONValue.h is included in the code using IsPointerConvertible<T,
> JSONValue>.

JSONValues.h is included by the generated InspectorTypeBuilder.h, which mainly
relies on implicit PassRefPtr<T> => PassRefPtr<JSONValue> convertibility.

[sof, 2015-03-11 08:25:26]

On 2015/03/11 06:24:44, sof wrote:
> On 2015/03/11 00:32:50, tkent wrote:
> > lgtm if JSONValue.h is included in the code using IsPointerConvertible<T,
> > JSONValue>.
> 
> JSONValues.h is included by the generated InspectorTypeBuilder.h, which mainly
> relies on implicit PassRefPtr<T> => PassRefPtr<JSONValue> convertibility.

Expanded the JSONValues.h comment to clarify why this kind of thing is needed.

[sof, 2015-03-11 09:48:19]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2015-03-11 09:48:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/991373002/#ps60001 (title:
"clarify why this specialization is needed")

[commit-bot: I haz the power, 2015-03-11 09:48:37]

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

[commit-bot: I haz the power, 2015-03-11 10:24:50]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191688