1320126 - Remove unused chrome://browser/skin/keyhole-forward-mask.svg file

Status: RESOLVED FIXED

(Firefox :: Theme, defect, P3)

Description

[Florian Quèze [:florian]]

Reported as unreferenced by the test I'm working on in bug 1316187.

This file was introduced in bug 544999 but the reference to it was commented out. The last (commented out) reference has been removed in bug 997131 (https://hg.mozilla.org/mozilla-central/rev/0ff00dde7e0c#l7.31).

Comment 1

[pass2pawan]

Hello i would like to patch this bug and similar ones in this dependency tree bug 1316187 (https://bugzilla.mozilla.org/showdependencytree.cgi?id=1316187&hide_resolved=1), but when i try to remove and then build firefox i get an error saying the same files are missing , have the references really been commented out ?

Comment 2

[Florian Quèze [:florian]]

(In reply to pass2pawan from comment #1)
> Hello i would like to patch this bug and similar ones in this dependency
> tree bug 1316187

Nice :-)

> when i try to remove and then build
> firefox i get an error saying the same files are missing , have the
> references really been commented out ?

Files that are packaged in chrome:// are referenced in a packaging manifest, a jar.mn file.

Here it's at http://searchfox.org/mozilla-central/rev/a712d69adb9b2588f88aff678216b2be94d3719c/browser/themes/windows/jar.mn#18

Have you removed that line when you tried building?

Comment 3

[pass2pawan]

Attachment: bug-1320126.patch

Comment 4

[Florian Quèze [:florian]]

Comment on attachment 8826179 [details] [diff] [review]
bug-1320126.patch

Looks good to me, thanks!

Comment 5

[Florian Quèze [:florian]]

I had to apply the change to jar.mn by hand because your tree doesn't contain this change yet https://hg.mozilla.org/mozilla-central/diff/5e444885af53/browser/themes/windows/jar.mn

Please update your tree before creating patches next time :-).

Comment 6

[Florian Quèze [:florian]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2815d3833df6436abf2f466a96204261135b6a81
Bug 1320126 - Remove unused chrome://browser/skin/keyhole-forward-mask.svg file, r=florian.

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2815d3833df6

Comment 8

[pass2pawan]

Hello , will i be assigned these bugs ? and what do you mean by updating my tree ?

Comment 9

[Florian Quèze [:florian]]

(In reply to pass2pawan from comment #8)
> Hello , will i be assigned these bugs ?

Yes, I thought it would happen automatically with the script resolving the bugs, but apparently it didn't.

> and what do you mean by updating my tree ?

Before starting new work, you need to do:
  hg pull
and
  hg update

The first command will download the new changes that were made by others, and the second will apply these changes to your local copy of the source code.