WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
51702
Web Inspector: add UglifyJS parser and formatter files
https://bugs.webkit.org/show_bug.cgi?id=51702
Summary
Web Inspector: add UglifyJS parser and formatter files
Pavel Podivilov
Reported
2010-12-29 05:25:01 PST
Web Inspector: add UglifyJS parser and formatter files (under BSD license this time).
Attachments
Patch.
(112.17 KB, patch)
2010-12-29 05:26 PST
,
Pavel Podivilov
pfeldman
: review-
Details
Formatted Diff
Diff
Patch.
(121.10 KB, patch)
2011-01-18 03:02 PST
,
Pavel Podivilov
no flags
Details
Formatted Diff
Diff
Move UglifyJS files to UglifyJS directory
(120.95 KB, patch)
2011-01-18 06:56 PST
,
Pavel Podivilov
yurys
: review+
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Pavel Podivilov
Comment 1
2010-12-29 05:26:04 PST
Created
attachment 77613
[details]
Patch.
Pavel Feldman
Comment 2
2010-12-30 06:17:42 PST
Comment on
attachment 77613
[details]
Patch. We should land this once there is a usage. It would also be great if we could make SourceJavaScriptTokenizer.js use this new utility. It would mitigate the footprint increase.
Pavel Podivilov
Comment 3
2011-01-18 03:02:35 PST
Created
attachment 79256
[details]
Patch.
Yury Semikhatsky
Comment 4
2011-01-18 05:57:14 PST
Comment on
attachment 79256
[details]
Patch. View in context:
https://bugs.webkit.org/attachment.cgi?id=79256&action=review
> Source/WebCore/inspector/front-end/scriptFormatterWorker.js:2 > + * Copyright (C) 2011 Google Inc. All rights reserved.
scriptFormatterWorker.js -> ScriptFormatterWorker.js
Yury Semikhatsky
Comment 5
2011-01-18 05:58:00 PST
UglifyJS sources should go into their own folder, otherwise looks good.
Pavel Podivilov
Comment 6
2011-01-18 06:56:47 PST
Created
attachment 79275
[details]
Move UglifyJS files to UglifyJS directory
Pavel Podivilov
Comment 7
2011-01-19 09:58:04 PST
Committed
r76132
: <
http://trac.webkit.org/changeset/76132
>
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