WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
70573
two inspector tests are failing after
r98052
https://bugs.webkit.org/show_bug.cgi?id=70573
Summary
two inspector tests are failing after r98052
Dirk Pranke
Reported
2011-10-20 19:18:14 PDT
the fixes for
bug 70483
appear to still have two tests failing on chromium, at least:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=inspector%2Fdebugger%2Flinkifier.html%2Cinspector%2Fdebugger%2Fscript-formatter.html
Attachments
Add attachment
proposed patch, testcase, etc.
Dirk Pranke
Comment 1
2011-10-20 19:24:22 PDT
Committed
r98056
: <
http://trac.webkit.org/changeset/98056
>
Dirk Pranke
Comment 2
2011-10-20 20:55:03 PDT
Note that
comment #1
is just for the test_expectations.txt suppression; the code is still broken. unfortunately, I got distracted by other things, and in between that and getting a build environment up to the point where I could attempt to debug this took too long, so I didn't make any useful progress. It's all you from here ...
Adam Barth
Comment 3
2011-10-20 21:04:38 PDT
Thanks Dirk. Will fix.
Adam Barth
Comment 4
2011-10-20 22:57:09 PDT
inspector/debugger/linkifier.html inspector/debugger/script-formatter.html
Adam Barth
Comment 5
2011-10-20 23:02:09 PDT
These tests are all skipped on Mac: BUGCR89357 SKIP MAC : inspector = PASS
Adam Barth
Comment 6
2011-10-20 23:03:24 PDT
This might have been fixed by
http://trac.webkit.org/changeset/98066
. Waiting for more bots to cycle to confirm.
Yuzo Fujishima
Comment 7
2011-10-20 23:53:04 PDT
On Chromium, the tests pass on Win and Linux but timeout on Mac (at least for 10.6). Changed the test expectation by
http://trac.webkit.org/changeset/98073
Adam Barth
Comment 8
2011-10-20 23:56:13 PDT
Removed timeout expectations in
http://trac.webkit.org/changeset/98076
because these tests are skipped on MAC.
Brian Burg
Comment 9
2014-12-17 16:42:37 PST
Test(s) have been removed.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug