Have Document::fonts() return a raw pointer

Have Document::fonts() return a raw pointer

Have Document::fonts() return a raw pointer instead of a PassRefPtr as
we are not passing ownership to the caller.

R=adamk
BUG=

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

[adamk, 2013-10-16 18:15:41]

lgtm

This makes me think we might want to consider adding bindings code that knows
how to handle references...

[Inactive, 2013-10-16 18:26:49]

On 2013/10/16 18:15:41, adamk wrote:
> lgtm
> 
> This makes me think we might want to consider adding bindings code that knows
> how to handle references...

Well, in theory, the attributes can already be marked as nullable or not in IDL
and we could support this easily in the generator. However, I don't think we are
ready for this. It would really like to get there though:

interface Test {
    attribute Node next; // Calls Node& Test::next()
    attribute Node? previous; // Calls Node* Test::previous()
};

[commit-bot: I haz the power, 2013-10-16 18:27:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/27544002/1

[adamk, 2013-10-16 18:40:14]

On 2013/10/16 18:26:49, Chris Dumez wrote:
> On 2013/10/16 18:15:41, adamk wrote:
> > lgtm
> > 
> > This makes me think we might want to consider adding bindings code that
knows
> > how to handle references...
> 
> Well, in theory, the attributes can already be marked as nullable or not in
IDL
> and we could support this easily in the generator. However, I don't think we
are
> ready for this. It would really like to get there though:
> 
> interface Test {
>     attribute Node next; // Calls Node& Test::next()
>     attribute Node? previous; // Calls Node* Test::previous()
> };

Oh, I wasn't even thinking of the IDL. The only thing preventing this one, for
example, from supporting references is generating a version of
v8SetReturnValueFast() that takes a FontFaceSet&. But maybe we should just leave
that till nbarth finishes our shiny new Python codegen.

[commit-bot: I haz the power, 2013-10-16 20:43:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/27544002/1

[commit-bot: I haz the power, 2013-10-17 01:37:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/27544002/1

[commit-bot: I haz the power, 2013-10-17 09:00:14]

Change committed as 159846