Bug 135177

Summary: REGRESSION (r171239): Much more time spent taking snapshots during the PLT
Product: WebKit Reporter: Tim Horton <thorton>
Component: WebKit2Assignee: Tim Horton <thorton>
Severity: Normal CC: commit-queue, mitz, rniwa, sam, simon.fraser
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 135190    
Bug Blocks:    
Description Flags
patch mitz: review+

Description Tim Horton 2014-07-22 14:37:35 PDT
We're taking way more snapshots now due to a mistake in r171239.

Comment 1 Tim Horton 2014-07-22 14:43:28 PDT
Created attachment 235316 [details]
Comment 2 Tim Horton 2014-07-22 14:47:11 PDT
Comment 3 Alexey Proskuryakov 2014-07-22 22:53:15 PDT
This change broke three API tests, so I'm going to roll out.

Tests that failed:
Tests that timed out:
Comment 4 WebKit Commit Bot 2014-07-22 23:00:02 PDT
Re-opened since this is blocked by bug 135190
Comment 5 Tim Horton 2014-07-23 03:04:54 PDT
What! How is that possible? I added this message very recently and the *only* thing it does is take a snapshot. Will investigate.
Comment 6 Tim Horton 2014-07-23 03:16:49 PDT
The UI process is stuck in a HashTable lock adding a string to the AtomicString table under decidePolicyForResponse. Still unclear how I could have influenced this.
Comment 7 Tim Horton 2014-07-23 03:19:03 PDT
Given http://trac.webkit.org/changeset/171367 I wonder if ap was off-by-one somehow. Testing now.
Comment 8 Tim Horton 2014-07-23 03:23:48 PDT
Rolling out my patch changes nothing, rolling out Joe's fixes all three tests. gg.
Comment 9 Tim Horton 2014-07-23 03:25:30 PDT
(In reply to comment #8)
> Rolling out my patch changes nothing, rolling out Joe's fixes all three tests. gg.

Rolled this one back in in http://trac.webkit.org/changeset/171471, will go roll Joe out.