Closed
Bug 1276537
Opened 9 years ago
Closed 4 years ago
Remove pre-Lion (10.7) fullscreen logic
Categories
(Firefox :: Toolbars and Customization, defect)
Firefox
Toolbars and Customization
Tracking
()
RESOLVED
FIXED
Firefox 78
People
(Reporter: MattN, Assigned: xidorn)
References
(Blocks 1 open bug)
Details
Attachments
(4 files)
Since support for 10.6-10.8 was dropped we can remove special code to implement custom fullscreen logic for 10.6 users which didn't have native fullscreen.
Some things I saw are:
* https://mxr.mozilla.org/mozilla-central/source/browser/themes/shared/browser.inc#7 (#restore-button)
* https://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.xul#976
but there is probably other places and it would be worth auditing browser-fullScreen.js.
Assignee | ||
Comment 1•5 years ago
|
||
Updated•5 years ago
|
Assignee: nobody → xidorn+moz
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•5 years ago
|
||
Depends on D74555
Assignee | ||
Comment 3•5 years ago
|
||
Depends on D74556
Assignee | ||
Comment 4•5 years ago
|
||
I didn't find much pre-lion code in the files mentioned in comment 0 actually. Maybe I'm missing something...
Anyway https://treeherder.mozilla.org/#/jobs?repo=try&revision=25f0e6742ee2c299a940344241a95dd17a10fb92 hope it wouldn't break anything this time...
Updated•5 years ago
|
Blocks: fx-fullscreen
Assignee | ||
Comment 6•5 years ago
|
||
Updated•5 years ago
|
Attachment #9147085 -
Attachment description: Bug 1276537 part 1 - Remove widget code for compatiblity with pre-Lion macOS. r?spohl → Bug 1276537 part 2 - Remove nsIWidget::SetShowsFullScreenButton. r?spohl
Updated•5 years ago
|
Attachment #9147086 -
Attachment description: Bug 1276537 part 2 - Remove OS X detection code from full screen handling. r?dao → Bug 1276537 part 3 - Remove OS X detection code from full screen handling. r?dao
Updated•5 years ago
|
Attachment #9147087 -
Attachment description: Bug 1276537 part 3 - Remove now-useless fullscreenbutton attribute. r?dao → Bug 1276537 part 4 - Remove now-useless fullscreenbutton attribute. r?dao
Pushed by mozilla@upsuper.org:
https://hg.mozilla.org/integration/autoland/rev/7b2be27b7bb9
part 1 - Make resizable window on macOS always show full screen button. r=spohl
https://hg.mozilla.org/integration/autoland/rev/6d7fac080dca
part 2 - Remove nsIWidget::SetShowsFullScreenButton. r=spohl
https://hg.mozilla.org/integration/autoland/rev/d809b87c8868
part 3 - Remove OS X detection code from full screen handling. r=dao
https://hg.mozilla.org/integration/autoland/rev/ea043dfff6a6
part 4 - Remove now-useless fullscreenbutton attribute. r=dao
Comment 8•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7b2be27b7bb9
https://hg.mozilla.org/mozilla-central/rev/6d7fac080dca
https://hg.mozilla.org/mozilla-central/rev/d809b87c8868
https://hg.mozilla.org/mozilla-central/rev/ea043dfff6a6
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox78:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 78
Updated•4 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•