Prevent implicit WebInputEvent construction from WebInputEvent::Type

Prevent implicit WebInputEvent construction from WebInputEvent::Type

All concrete WebInputEvent derivatives currently accept an unsigned size
parameter in the constructor, used to zero-initialize the object.  This can
lead to unexpected runtime behavior, e.g., a WebInputEvent::Type value can
implicitly construct a WebInputEvent object.  Prevent this by making the
size-accepting constructor protected for all types, and preventing naked
construction of a base WebInputEvent instance.

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

[jdduke (slow), 2014-04-01 21:30:48]

tkent@: PTAL.  This issue has cropped up every now and then, and while there
shouldn't be any current violations, best to prevent such in the future.

[tkent, 2014-04-02 00:18:49]

lgtm

https://codereview.chromium.org/132183008/diff/80001/public/web/WebInputEvent.h
File public/web/WebInputEvent.h (right):

https://codereview.chromium.org/132183008/diff/80001/public/web/WebInputEvent...
public/web/WebInputEvent.h:230: WebInputEvent();
Is this necessary?

[jdduke (slow), 2014-04-02 20:20:59]

https://codereview.chromium.org/132183008/diff/80001/public/web/WebInputEvent.h
File public/web/WebInputEvent.h (right):

https://codereview.chromium.org/132183008/diff/80001/public/web/WebInputEvent...
public/web/WebInputEvent.h:230: WebInputEvent();
On 2014/04/02 00:18:52, tkent wrote:
> Is this necessary?

Nope, removed.

[jdduke (slow), 2014-04-02 20:21:03]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-04-02 20:21:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/132183008/100001

[commit-bot: I haz the power, 2014-04-02 20:31:30]

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

[commit-bot: I haz the power, 2014-04-02 20:31:30]

Try jobs failed on following builders:
  tryserver.blink on win_blink_compile_dbg

[jdduke (slow), 2014-04-02 21:42:42]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-04-02 21:42:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/132183008/100001

[commit-bot: I haz the power, 2014-04-02 22:11:13]

Change committed as 170708