-
-
Notifications
You must be signed in to change notification settings - Fork 216
Pull upstream #1840
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Pull upstream #1840
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
IGNORE BROKEN CHANGESETS CLOSED TREE
Original Revision: https://phabricator.services.mozilla.com/D253675 Differential Revision: https://phabricator.services.mozilla.com/D255155
…ixes - a=diannaS Differential Revision: https://phabricator.services.mozilla.com/D255479
… yield. a=pascalc Original Revision: https://phabricator.services.mozilla.com/D255317 Differential Revision: https://phabricator.services.mozilla.com/D255429
…. r=#gfx-reviewers a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D253819
…=emz a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D235334
…en we're using the system UI (rather than printing via the Firefox preview panel). a=pascalc The patch in bug 1898184 made reasonable sense, inasmuch as when the print.prefer_system_dialog option is set, we don't show our window-modal preview panel. However, as noted in comment 6 here, we do still need to block JS while window.print() is presenting the system dialog, otherwise the document may be changed or even closed by script behind the dialog's back. Prior to bug 1898184, the IsPreview::Yes flag caused us to block, but the issue was that without the preview panel taking care of cleaning up the cloned document, the block got cleared. It looks like we can solve this by setting the closeWindowAfterPrint flag in this case. In local testing, this seems to work for the various examples here, and does not regress bug 1898184. Original Revision: https://phabricator.services.mozilla.com/D253655 Differential Revision: https://phabricator.services.mozilla.com/D253945
…ixes - a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D256260
…devtools-reviewers,nchevobbe a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D254323
…or #strip-url-for-use-in-reports. r=freddyb a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D256232
…ialize for moz2d recording IPC. r=gfx-reviewers,lsalzman a=pascalc Original Revision: https://phabricator.services.mozilla.com/D244105 Differential Revision: https://phabricator.services.mozilla.com/D252887
…ent. r=smaug a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D255478
… doesn't exist in esr128 r=gfx-reviewers,jrmuizel a=RyanVM Remove some code for clearing the mozilla::layers::SurfaceDescriptorMacIOSurface gpuFence member, this field (and the concept it is for) didn't exist in esr128. Differential Revision: https://phabricator.services.mozilla.com/D256734
Ion stores stack results that are refs using MWasmStoreRef, and this is only safe because we manually disable the barriers for the instruction. Add a dedicated node that is safe for ref stack results and just does the plain store we need. Original Revision: https://phabricator.services.mozilla.com/D255036 Differential Revision: https://phabricator.services.mozilla.com/D256703
…er which doesn't exist in esr128 r=gfx-reviewers,jrmuizel a=RyanVM" for bustage a=backout This reverts commit c83064a.
…s we serialize for moz2d recording IPC. r=gfx-reviewers,lsalzman a=pascalc" for bustage a=backout This reverts commit cff14db.
…eting an entry from the table cache. a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D256851
… a=RyanVM Differential Revision: https://phabricator.services.mozilla.com/D257002
…ixes - a=repo-update Differential Revision: https://phabricator.services.mozilla.com/D256788
…128). r=#search-reviewers a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D257016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Warning: Be sure to create a merge commit and merge it in.