WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
144412
Web Inspector: Remove some unused Images
https://bugs.webkit.org/show_bug.cgi?id=144412
Summary
Web Inspector: Remove some unused Images
Timothy Hatcher
Reported
2015-04-29 15:26:57 PDT
Images we don't reference anywhere.
Attachments
Patch
(30.18 KB, patch)
2015-04-29 15:32 PDT
,
Timothy Hatcher
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Timothy Hatcher
Comment 1
2015-04-29 15:32:01 PDT
Created
attachment 251994
[details]
Patch
Joseph Pecoraro
Comment 2
2015-04-29 16:40:22 PDT
Comment on
attachment 251994
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=251994&action=review
r=me
> Source/WebInspectorUI/UserInterface/Images/Reload.svg:-1 > -<?xml version="1.0" encoding="utf-8"?>
Interesting! Maybe we should rename "ReloadFull.svg" to "Reload.svg".
WebKit Commit Bot
Comment 3
2015-04-29 17:29:19 PDT
Comment on
attachment 251994
[details]
Patch Clearing flags on attachment: 251994 Committed
r183594
: <
http://trac.webkit.org/changeset/183594
>
WebKit Commit Bot
Comment 4
2015-04-29 17:29:24 PDT
All reviewed patches have been landed. Closing bug.
Andres Gomez Garcia
Comment 5
2015-04-30 03:51:36 PDT
(In reply to
comment #0
)
> Images we don't reference anywhere.
Hei, Timothy! On future removals, don't be shy and clean also any image in the gtk subdirectory if it is not needed any more :) I will remove them now, as part of the checks I'm making.
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