Make InputType and InputTypeView RefCounted.

Make InputType and InputTypeView RefCounted.

This is a preparation to add HTMLInputElement::m_inputTypeView. Both of
m_inputType and m_inputTypeView will be able to own single InputType object.

BUG=274205
TEST=none; no behavior changes.

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

[esprehn, 2013-08-27 02:26:25]

What's the benefit of them being RefCounted? This makes all InputTypes larger.

[tkent, 2013-08-27 03:24:30]

On 2013/08/27 02:26:25, esprehn wrote:
> What's the benefit of them being RefCounted? This makes all InputTypes larger.

We need this change because an HTMLInputElement will have two modes

a) HTMLInputElement::m_inputType owns an InputType object, and
HTMLInputElement::m_inputTypeView also owns the same InputType object.
  An InputType object works as an InputTypeView for now.
b) m_inputType owns an InputType object, and m_inputTypeView owns an
InputTypeView object for author shadow DOM case.

RefCounted InputType handles it smartly.
a) An InputType object has two reference counts.
b) An InputType object has one reference count, and an InputTypeView object has
one reference count.


I think this is a temporary change.  Implementations of InputType class contain
InputTypeView for now. I'd like to make separated InputTypeView classes for all
of input types. We  will be able to stop the RefCounted InputType at that
moment.

[keishi, 2013-08-27 16:39:09]

lgtm

[esprehn_google.com, 2013-08-27 17:01:11]

Lgtm

On Tuesday, August 27, 2013, wrote:

> lgtm
>
>
>
>
https://codereview.chromium.**org/23502003/<https://codereview.chromium.org/2...
>
> To unsubscribe from this group and stop receiving emails from it, send an
> email to blink-reviews+unsubscribe@chromium.org.
>

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-08-27 23:42:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tkent@chromium.org/23502003/1

[commit-bot: I haz the power, 2013-08-28 00:01:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tkent@chromium.org/23502003/1

[commit-bot: I haz the power, 2013-08-28 00:17:53]

Change committed as 156800