Don't force layout for mouse event hit tests

Don't force layout for mouse event hit tests

Second try. This time with a different approach to deal with tests. See
the EventSender change (this kinda reverts r150324).

Before the initial document layout, we shouldn't force a layout for
mouse events, because they are not intended for the new document anyway -
the user can't see the document yet! Just ignore those events by making
the hit tests match nothing.

Otherwise, the hit test would force a layout, which is undesirable
because that leads to a flash of an empty document.

BUG=1373
R=ojan@chromium.org

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

[ojan, 2013-05-21 02:12:37]

lgtm

https://codereview.chromium.org/15425005/diff/1/Tools/DumpRenderTree/chromium...
File Tools/DumpRenderTree/chromium/TestRunner/src/EventSender.cpp (right):

https://codereview.chromium.org/15425005/diff/1/Tools/DumpRenderTree/chromium...
Tools/DumpRenderTree/chromium/TestRunner/src/EventSender.cpp:431:
webview()->layout();
Nit: Could move this into a helper function, EventSender::layoutWebview() or
something to avoid code duplication and make it more likely that new events also
respect this bool. Not a big deal.

[cbiesinger, 2013-05-21 02:19:58]

On 2013/05/21 02:12:37, ojan wrote:
> lgtm
> 
>
https://codereview.chromium.org/15425005/diff/1/Tools/DumpRenderTree/chromium...
> File Tools/DumpRenderTree/chromium/TestRunner/src/EventSender.cpp (right):
> 
>
https://codereview.chromium.org/15425005/diff/1/Tools/DumpRenderTree/chromium...
> Tools/DumpRenderTree/chromium/TestRunner/src/EventSender.cpp:431:
> webview()->layout();
> Nit: Could move this into a helper function, EventSender::layoutWebview() or
> something to avoid code duplication and make it more likely that new events
also
> respect this bool. Not a big deal.

I think I'll keep it without the helper function because I don't think it helps
with new events. The reasoning is: if someone adds a new event, they will only
notice the helper function if they look at the existing code. But if they look
at the existing code with the explicit ifs, they'll notice the bool anyway.

Will submit once I have try results...

[cbiesinger, 2013-05-21 22:38:29]

Committed patchset #3 manually as r150837 (presubmit successful).