Remove implementation of Entity constructor

Remove implementation of Entity constructor

Remove implementation of Entity constructor as we never construct Entity
objects. This will lead in a link time error if someone tries to use it
instead of a runtime error (due to the ASSERT_NOT_REACHED() in the
constructor).

This gets rid of a case where 0 is passed as TreeScope to the ContainerNode /
Node constructor.

R=adamk

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

[eseidel, 2013-11-14 16:15:56]

I'm confused.  Why do we even have this if we can't construct one?

Is this just to support compiling of Entity.idl (which presumably we to expose
to the page for some reason?)

[Inactive, 2013-11-14 17:15:18]

On 2013/11/14 16:15:56, eseidel wrote:
> I'm confused.  Why do we even have this if we can't construct one?
> 
> Is this just to support compiling of Entity.idl (which presumably we to expose
> to the page for some reason?)

For historical reasons, we have Entity.idl and we expose window.Entity to JS. We
need this code for building successfully. However, we should never construct an
Entity object. Having the constructor private and unimplemented ensures that.

We can get rid of all this once we get rid of Entity.idl.

[eseidel, 2013-11-14 17:30:26]

lgtm

I suspect we can go further... but OK.

[commit-bot: I haz the power, 2013-11-14 17:32:12]

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

[commit-bot: I haz the power, 2013-11-14 19:01:29]

Change committed as 162041