We currently skip ALL inspector layout tests, to avoid assertions which occur during a handful of tests (https://bugs.webkit.org/show_bug.cgi?id=129642). Recommend skipping only tests in inspector/replay.
<rdar://problem/18170451>
TestExpectations already had a bug covering skipping all inspector tests: > webkit.org/b/129642 inspector [ Skip ] <https://webkit.org/b/129642> StorageTracker::deleteOrigin being called off the main thread (ASSERTs in inspector/test-harness-trivially-works.html test)
Created attachment 237531 [details] Patch
Assertions still occur outside of inspector/replay, so for now we'll continue to skip all inspector tests. This issue is intended to address a couple unrelated bugs which were discovered while investigating https://bugs.webkit.org/show_bug.cgi?id=129642.
Comment on attachment 237531 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=237531&action=review r=me, update the ChangeLog with the new bug title before landing. > LayoutTests/ChangeLog:9 > + Updated expected results to reflect changes to breakpoint resolution in > + issue: https://bugs.webkit.org/show_bug.cgi?id=135396. Instead of referencing the bug, you should reference the r####.
Created attachment 237534 [details] Patch
Comment on attachment 237534 [details] Patch r=me
Comment on attachment 237534 [details] Patch Clearing flags on attachment: 237534 Committed r173405: <http://trac.webkit.org/changeset/173405>
All reviewed patches have been landed. Closing bug.