Use [ImplementedAs=domWindow] for Document.defaultView()

Use [ImplementedAs=domWindow] for Document.defaultView()

Use [ImplementedAs=domWindow] for Document.defaultView() to avoid having 2
implementation methods that return the same thing. This makes our
implementation more consistent.

R=haraken
BUG=

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

[Inactive, 2013-10-14 21:42:50]

+abarth for Source/web

[arv (Not doing code reviews), 2013-10-14 21:49:47]

I would prefer that we got rid of domWindow instead.

[Inactive, 2013-10-14 22:35:04]

On 2013/10/14 21:49:47, arv wrote:
> I would prefer that we got rid of domWindow instead.

I thought that domWindow() was used a lot more in our implementation code but
maybe my assumption was wrong. I also felt what was returned by the method was
clearer but maybe it's just me.

[haraken, 2013-10-15 00:18:52]

On 2013/10/14 22:35:04, Chris Dumez wrote:
> On 2013/10/14 21:49:47, arv wrote:
> > I would prefer that we got rid of domWindow instead.
> 
> I thought that domWindow() was used a lot more in our implementation code but
> maybe my assumption was wrong. I also felt what was returned by the method was
> clearer but maybe it's just me.

I'd prefer domWindow() rather than defaultView() for clarity. LGTM if arv is OK
with this change.

[arv (Not doing code reviews), 2013-10-15 00:48:20]

I always tend to prefer sticking to the names in the spec for least confusion
but one name is better than 2.

LGTM

[commit-bot: I haz the power, 2013-10-15 13:16:46]

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

[commit-bot: I haz the power, 2013-10-15 13:29:55]

Retried try job too often on blink_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_pres...

[Inactive, 2013-10-15 15:37:19]

On 2013/10/15 13:29:55, I haz the power (commit-bot) wrote:
> Retried try job too often on blink_presubmit for step(s) presubmit
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_pres...

abarth, could you please take a look for Source/web?

[abarth-chromium, 2013-10-15 18:29:07]

Source/web <--- LGTM

[commit-bot: I haz the power, 2013-10-15 18:30:05]

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

[commit-bot: I haz the power, 2013-10-16 03:59:49]

Change committed as 159715