Bug 55690 – Web Inspector: install onerror handler in inspector tests

• Component: Web Inspector (Deprecated)
• Status: RESOLVED FIXED

[Andrey Kosyakov]

Currently, web inspector console is not captured in test output. We redefine console.log & co in inspector test suites to direct the messages to test output, yet if an exception happens in front-end during test, we do not capture the exception information.
This patch adds onerror handler that will post error information to test output.

[Andrey Kosyakov]

Created attachment 84597 [details]
patch

[WebKit Commit Bot]

Comment on attachment 84597 [details]
patch

Rejecting attachment 84597 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-8', 'apply-..." exit_code: 2

Last 500 characters of output:
ed 3 diffs from patch file(s).
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/http/tests/inspector/inspector-test.js
Hunk #1 FAILED at 74.
1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/http/tests/inspector/inspector-test.js.rej
patching file LayoutTests/http/tests/inspector/resources/extension-main.js

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Pavel Feldman', u'--fo..." exit_code: 1

Full output: http://queues.webkit.org/results/8038025

[Andrey Kosyakov]

Manually committed r80417: http://trac.webkit.org/changeset/80417

[Andrey Kosyakov]

Comment on attachment 84597 [details]
patch

Removing pfeldman's r+ from obsolete patch.