Make ancestors of modal <dialog> inert

Make ancestors of modal <dialog> inert

As per spec change: http://html5.org/r/8343

Now, document.body is also inert. Previously events that
"skipped" an inert node were fired on document.body; now,
they are fired on document.

Test: modal-dialog-ancestor-is-inert.html
Also, other tests cover some of this behavior and have
expectations updated.

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

[falken, 2014-01-10 11:45:05]

Elliott, could you review please?

[esprehn, 2014-01-14 01:43:29]

lgtm

https://codereview.chromium.org/133393002/diff/60001/LayoutTests/fast/dom/HTM...
File
LayoutTests/fast/dom/HTMLDialogElement/modal-dialog-blocks-mouse-events.html
(right):

https://codereview.chromium.org/133393002/diff/60001/LayoutTests/fast/dom/HTM...
LayoutTests/fast/dom/HTMLDialogElement/modal-dialog-blocks-mouse-events.html:81:
document.expectedEventCount = events.length - 1; // document won't get
'mouseout'
Why are you storing these properties on the document and now in the window? It's
really weird to add these expandos.

[falken, 2014-01-14 01:50:47]

https://codereview.chromium.org/133393002/diff/60001/LayoutTests/fast/dom/HTM...
File
LayoutTests/fast/dom/HTMLDialogElement/modal-dialog-blocks-mouse-events.html
(right):

https://codereview.chromium.org/133393002/diff/60001/LayoutTests/fast/dom/HTM...
LayoutTests/fast/dom/HTMLDialogElement/modal-dialog-blocks-mouse-events.html:81:
document.expectedEventCount = events.length - 1; // document won't get
'mouseout'
On 2014/01/14 01:43:29, esprehn wrote:
> Why are you storing these properties on the document and now in the window?
It's
> really weird to add these expandos.

Seemed a bit nicer to say document.expectedEventCount than
expectedEventCountForDocument etc, but I'll move them to window.

[commit-bot: I haz the power, 2014-01-14 04:50:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/falken@chromium.org/133393002/220001

[commit-bot: I haz the power, 2014-01-14 08:09:10]

Change committed as 165040

[tdanderson, 2014-01-14 18:24:30]

A revert of this CL has been created in
https://codereview.chromium.org/138183003/ by tdanderson@chromium.org.

The reason for reverting is: This patch is the most probable cause of the
failing content_browsertests:

AccessibilityModalDialogInIframeOpened
AccessibilityModalDialogOpened
AccessibilityModalDialogStack

See:

http://build.chromium.org/p/chromium.webkit/builders/Win7%20Tests/builds/529

http://build.chromium.org/p/chromium.webkit/builders/Win7%20%28dbg%29/builds/...

http://build.chromium.org/p/chromium.webkit/builders/Mac10.6%20Tests/builds/2...

http://build.chromium.org/p/chromium.webkit/builders/Mac10.8%20Tests/builds/5388

I could not reach the patch author on IRC due to the time zone difference..

[falken, 2014-01-15 01:18:40]

On 2014/01/14 18:24:30, tdanderson wrote:
> A revert of this CL has been created in
> https://codereview.chromium.org/138183003/ by mailto:tdanderson@chromium.org.
> 
> The reason for reverting is: This patch is the most probable cause of the
> failing content_browsertests:
> 
> AccessibilityModalDialogInIframeOpened
> AccessibilityModalDialogOpened
> AccessibilityModalDialogStack
> 
> See:
> 
> http://build.chromium.org/p/chromium.webkit/builders/Win7%2520Tests/builds/529
> 
>
http://build.chromium.org/p/chromium.webkit/builders/Win7%2520%2528dbg%2529/b...
> 
>
http://build.chromium.org/p/chromium.webkit/builders/Mac10.6%2520Tests/builds...
> 
>
http://build.chromium.org/p/chromium.webkit/builders/Mac10.8%2520Tests/builds...
> 
> I could not reach the patch author on IRC due to the time zone difference..

Sorry, I forgot these tests will need new expectations. This will have to be a
3-sided patch.