Bug 80166 - Web Inspector: InspectorCounters mechanism should be thread-safe
Summary: Web Inspector: InspectorCounters mechanism should be thread-safe
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Yury Semikhatsky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-02 09:06 PST by Yury Semikhatsky
Modified: 2012-03-06 08:20 PST (History)
13 users (show)

See Also:


Attachments
Patch (12.26 KB, patch)
2012-03-02 09:08 PST, Yury Semikhatsky
no flags Details | Formatted Diff | Diff
Patch (12.45 KB, patch)
2012-03-05 06:05 PST, Yury Semikhatsky
no flags Details | Formatted Diff | Diff
Patch (14.98 KB, patch)
2012-03-06 07:54 PST, Yury Semikhatsky
pfeldman: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yury Semikhatsky 2012-03-02 09:06:33 PST
There are some counters which make sense for workers as well (e.g. event listener count) and since workers run in their own threads it should be safe to increment/decrement counters from there.
Comment 1 Yury Semikhatsky 2012-03-02 09:08:57 PST
Created attachment 129910 [details]
Patch
Comment 2 Yury Semikhatsky 2012-03-02 09:10:52 PST
Dromaeo results for Chromium run
with the patch applied: http://dromaeo.com/?id=164681
and without it: http://dromaeo.com/?id=164673
Comment 3 Early Warning System Bot 2012-03-02 09:23:04 PST
Comment on attachment 129910 [details]
Patch

Attachment 129910 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/11802009
Comment 4 Build Bot 2012-03-02 09:28:08 PST
Comment on attachment 129910 [details]
Patch

Attachment 129910 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/11797038
Comment 5 Pavel Feldman 2012-03-02 09:29:59 PST
(In reply to comment #2)
> Dromaeo results for Chromium run
> with the patch applied: http://dromaeo.com/?id=164681
> and without it: http://dromaeo.com/?id=164673

So you are saying it got faster?

Sounds like you are breaking ews.
Comment 6 Pavel Feldman 2012-03-02 09:30:34 PST
Comment on attachment 129910 [details]
Patch

r- for breaking build.
Comment 7 Yury Semikhatsky 2012-03-05 01:07:12 PST
(In reply to comment #5)
> (In reply to comment #2)
> > Dromaeo results for Chromium run
> > with the patch applied: http://dromaeo.com/?id=164681
> > and without it: http://dromaeo.com/?id=164673
> 
> So you are saying it got faster?
> 
Of cause not, it just one of the sampling errors. In two other runs the results were about the same:
http://dromaeo.com/?id=164676
http://dromaeo.com/?id=164678

> Sounds like you are breaking ews.
Will fix.
Comment 8 Yury Semikhatsky 2012-03-05 06:05:30 PST
Created attachment 130107 [details]
Patch
Comment 9 Yury Semikhatsky 2012-03-06 07:54:12 PST
Created attachment 130385 [details]
Patch
Comment 10 Yury Semikhatsky 2012-03-06 08:20:58 PST
Committed r109922: <http://trac.webkit.org/changeset/109922>