Closed Bug 714743 Opened 13 years ago Closed 10 years ago

[Firefox] removed-files.in: is full-screen-override.css removal actually (always) needed?

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla27

People

(Reporter: sgautherie, Unassigned)

References

()

Details

Bug 702146 added this unconditional removal.
Is it needed at all?
Should it rather be in '#ifdef MOZ_OMNIJAR'?
Blocks: 714744
I have no idea.  Someone familiar with the packaging stuff would be better to ask.
(In reply to Serge Gautherie (:sgautherie) from comment #0)

> Bug 702146 added this unconditional removal.

https://hg.mozilla.org/mozilla-central/rev/bb3bfc552568

> Is it needed at all?

full-screen-override.css is a new file,
and is packaged inside (layout/style/jar.mn >) toolkit.jar.
Hence, there is no reason to ever remove it, is it?

> Should it rather be in '#ifdef MOZ_OMNIJAR'?

That would probably rather be "if not flat", but I don't think we care about that.
Is this still actual?
Bump?
(Phoenix, sorry for the (2-year) delay!)

It seems this bug was still actual,
but bug 649607 eventually removed the "full-screen-override.css" line.

R.Fixed
Status: NEW → RESOLVED
Closed: 10 years ago
Depends on: 649607
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.