Bug 60669 – After relaunching an unresponsive web process, web page stops responding to mouse movement

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit2

[Brian Weinstein]

After relaunching an unresponsive web process, web page stops responding to mouse movement 

You can't can't scroll by dragging scrollbar, can't select text, etc.

<rdar://problem/9409934>

[Brian Weinstein]

Created attachment 93221 [details]
[PATCH] Fix with test

[Brian Weinstein]

Sam stated this test will take too long. Waiting for the responsiveness callback is too long to wait for this test.

[Adam Roben (:aroben)]

Comment on attachment 93221 [details]
[PATCH] Fix with test

r-ing on Sam's behalf.

[Brian Weinstein]

Created attachment 93297 [details]
[PATCH] Fix with faster test

[Anders Carlsson]

Comment on attachment 93297 [details]
[PATCH] Fix with faster test

View in context: https://bugs.webkit.org/attachment.cgi?id=93297&action=review

> Tools/TestWebKitAPI/Tests/WebKit2/MouseMoveAfterCrash.cpp:84
> +    // Pause will hang the WebProcess long enough to get the processDidBecomeUnresponsive call, and there, we
> +    // reload the page, which restarts the web process. Wait until the page has been reloaded to continue with the test.

This isn't true anymore, is it?

[WebKit Review Bot]

Attachment 93297 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1

Tools/TestWebKitAPI/Tests/WebKit2/MouseMoveAfterCrash.cpp:75:  Tab found; better to use spaces  [whitespace/tab] [1]
Tools/TestWebKitAPI/Tests/WebKit2/MouseMoveAfterCrash.cpp:79:  Tab found; better to use spaces  [whitespace/tab] [1]
Tools/TestWebKitAPI/Tests/WebKit2/MouseMoveAfterCrash.cpp:80:  Tab found; better to use spaces  [whitespace/tab] [1]
Tools/TestWebKitAPI/Tests/WebKit2/MouseMoveAfterCrash.cpp:81:  Tab found; better to use spaces  [whitespace/tab] [1]
Total errors found: 4 in 13 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Brian Weinstein]

Comment on attachment 93297 [details]
[PATCH] Fix with faster test

View in context: https://bugs.webkit.org/attachment.cgi?id=93297&action=review

>> Tools/TestWebKitAPI/Tests/WebKit2/MouseMoveAfterCrash.cpp:84
>> +    // reload the page, which restarts the web process. Wait until the page has been reloaded to continue with the test.
> 
> This isn't true anymore, is it?

Nope. I'll clean up the comment.

[Darin Adler]

r86354

[Simon Fraser (smfr)]

This test is timing out:
http://build.webkit.org/builders/SnowLeopard%20Intel%20Release%20%28WebKit2%20Tests%29/builds/15994/steps/run-api-tests/logs/stdio