WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
54010
Web Inspector: script formatter should not create a worker for each script.
https://bugs.webkit.org/show_bug.cgi?id=54010
Summary
Web Inspector: script formatter should not create a worker for each script.
Pavel Podivilov
Reported
2011-02-08 09:05:08 PST
Reusing a worker results in a much better performance when formatting multiple scripts inlined in html.
Attachments
Patch.
(2.55 KB, patch)
2011-02-08 09:06 PST
,
Pavel Podivilov
yurys
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Pavel Podivilov
Comment 1
2011-02-08 09:06:01 PST
Created
attachment 81644
[details]
Patch.
Yury Semikhatsky
Comment 2
2011-02-08 23:46:07 PST
Comment on
attachment 81644
[details]
Patch. View in context:
https://bugs.webkit.org/attachment.cgi?id=81644&action=review
> Source/WebCore/inspector/front-end/ScriptFormatter.js:133 > + this._lastTask = task;
I'd rather use and array with its push/shift methods
Pavel Podivilov
Comment 3
2011-02-09 02:12:34 PST
Committed
r78034
: <
http://trac.webkit.org/changeset/78034
>
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