Bug 144412 - Web Inspector: Remove some unused Images
Summary: Web Inspector: Remove some unused Images
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Timothy Hatcher
URL:
Keywords: DoNotImportToRadar
Depends on:
Blocks:
 
Reported: 2015-04-29 15:26 PDT by Timothy Hatcher
Modified: 2015-04-30 03:51 PDT (History)
9 users (show)

See Also:

Attachments
Patch (30.18 KB, patch)
2015-04-29 15:32 PDT, Timothy Hatcher 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Hatcher 2015-04-29 15:26:57 PDT 
Images we don't reference anywhere.
Comment 1 Timothy Hatcher 2015-04-29 15:32:01 PDT 
Created attachment 251994 [details]
Patch
Comment 2 Joseph Pecoraro 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".
Comment 3 WebKit Commit Bot 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>
Comment 4 WebKit Commit Bot 2015-04-29 17:29:24 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 5 Andres Gomez Garcia 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.