See https://bugzilla.mozilla.org/show_bug.cgi?id=1365614 for web compat issues this causes.
<!-- Please describe your changes on the following line: -->
---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix https://bugzilla.mozilla.org/show_bug.cgi?id=1365614
<!-- Either: -->
- [ ] There are tests for these changes OR
- [ ] These changes do not require tests because _____
<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->
<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
Source-Repo: https://github.com/servo/servo
Source-Revision: e9c5860808dec3a7ef5dd3a64179b249a2cbe9a4
--HG--
extra : subtree_source : https%3A//hg.mozilla.org/projects/converted-servo-linear
extra : subtree_revision : 42699696ca7326e9480c76dc44293458f1fc780a
Activity switching is now handled by the currently active GeckoApp instance, which obviously doesn't work if all of them are currently backgrounded.
To fix this, we track for each tab whether its selection was handled by a foregrounded GeckoApp instance. If it wasn't, we catch up with any possibly necessary activity switches during the next resume.
MozReview-Commit-ID: JEjQUuDJw5Q
--HG--
extra : rebase_source : b7a9f5a4ad4811c1c9e9d24eaae61f3757c64442
Normally, receiving a new intent means we want to switch tabs/open a new tab, so we need to turn off our "last selected tab in this activity" logic, however this is not true when we receive a simple Intent.ACTION_MAIN, e.g. when we're being brought into the foreground from a launcher icon (as opposed to the activity/task switcher).
MozReview-Commit-ID: ExiM3c8iaEA
--HG--
extra : rebase_source : c2c60a07c1cbbd6d3e1d16557721e975ae932a55
Switching between different GeckoApp instances in response to tab selection no longer requires the GeckoActivityMonitor for getting the current activity, so we no longer need to manually set the current activity for the benefit of code that executes early during onCreate/onNewIntent.
MozReview-Commit-ID: E9BQhRclDfS
--HG--
extra : rebase_source : 7fd279d5450c95d00d1df93b84b7a0679f686305
To avoid the need for tracking the currently active activity, we just respond from the activity itself by using GeckoApp's tab event listener.
As replacement for the "closeTabNoActivitySwitch()" function, we set a flag on the activity instead before closing the tab.
MozReview-Commit-ID: LdmyJ5i10jZ
--HG--
extra : rebase_source : c960b58eae3c862f89fef188022f8d71c88f06c6
These methods already take a tab object as their argument, so we just turn them into Tab member functions.
MozReview-Commit-ID: 7fp7bzDbRja
--HG--
extra : rebase_source : 801ee697138cdda8f82402583ee5348b74625197
We want to move the activity switching code into the activity (i.e. GeckoApp) itself and run it from its onTabChanged handler in response to the appropriate events (at the moment that's only SELECTED), so we need to ensure that this will actually be called within custom tabs/web apps as well.
Additionally, there's no need to separately register the tab events listener from the CustomTabs/WebAppActivity as well if our parent class already does it for us.
MozReview-Commit-ID: 6PIq1KncDcA
--HG--
extra : rebase_source : ba6448d214b26f966bccf1deb6f87c931dac2ec1
We recently accidentally broke the "Enable DRM" prompt without realising, so
this adds a test that if EME is disabled and is attempted to be used we we
prompt the user to enable EME.
It is important we don't regress this feature as without it users with DRM
disabled (which is our Linux popuation by default) may try to playback DRM
protected video and it won't work and there won't be an obvious way to make it
work.
MozReview-Commit-ID: D4X7D4wkwRy
--HG--
extra : rebase_source : d0c7bf45c0a3b4ffb517a3b7c008230bd750fc8d
DevTools are moving out of mozilla central, and since this test is relying on scratchpad
being available, it would probably make sense to move it to the scratchpad test suite
which will still be run in the new devtools continuous integration setup.
MozReview-Commit-ID: 19x6Ccp85ND
--HG--
rename : browser/components/sessionstore/test/browser_644409-scratchpads.js => devtools/client/scratchpad/test/browser_scratchpad_sessions.js
extra : rebase_source : bf564025199748b43ef56333f5c5626b266be1ca
AFAICT SimpleTest.registerCleanup is never called here. And it introduced a leak
in the session scratchpad test I want to move to this test suite. I couldn't really
understand where the leak was coming from, but using registerCleanup (which _is_
called) fixes it.
MozReview-Commit-ID: 2VE7wHxy066
--HG--
extra : rebase_source : b2e7f62a727535912be55c68d6f8b0932e9f1414
Windows repackaging for complete mar files will also need to pull a
different value out of the application.ini file, so this code should be
shareable.
MozReview-Commit-ID: CzCoNRYcBPX
--HG--
extra : rebase_source : a5e4b31ba876d811436a7d8d15462fa85f0762f8
The chmod permissions need to be in octal format to get the expected
permissions settings. This only seems to affect the output if we run the
repackaging command on Linux, but it should still be fixed.
MozReview-Commit-ID: to4v7dkSBl
--HG--
extra : rebase_source : 9d2289511ebd05aea8a4c6d37136f258e2463dee
Python can run these files with 'python -m 7z_exe_foo', but using
'import 7z_exe_foo' is not allowed because the module begins with a
number.
See also: https://stackoverflow.com/a/17487228
MozReview-Commit-ID: 97iDdXlZJ1a
--HG--
rename : python/mozbuild/mozbuild/action/7z_exe_archive.py => python/mozbuild/mozbuild/action/exe_7z_archive.py
rename : python/mozbuild/mozbuild/action/7z_exe_extract.py => python/mozbuild/mozbuild/action/exe_7z_extract.py
extra : rebase_source : 1941986a7e6e56305b742710c73b90a3f7b5226b
It makes more sense to have the repackage commands in a separate
directory, since Windows repackaging will add several new types.
MozReview-Commit-ID: 1wA7F7k4NXf
--HG--
rename : python/mozbuild/mozbuild/repackage.py => python/mozbuild/mozbuild/repackaging/dmg.py
extra : rebase_source : b4b4f2fd5c45900aaf125928c144c15cfa1e115e
Add support for asm.js global variables with the following types:
Int8x16, Bool8x16, Int16x8, Bool16x8.
We already have the needed code generation support, but tests were
missing for these types, and so their types were omitted from some
critical switches.
MozReview-Commit-ID: B4r7VofjlYL
--HG--
extra : rebase_source : 4df72f2296f814a1ea83d6ff93170ed2049f4361
This patch should not result any visual differences. Rather, it makes
nsTextPaintStyle::InitCommonColors() think the checkboxes and radio
buttons are not themed and it should calculate contrast properly.
MozReview-Commit-ID: LF1bQQE1V9X
--HG--
extra : rebase_source : 38c04fa02c97779b60ca55697bc1ecbc37556d1b
This event made us initially fade out the URL bar text in new windows despite the text not overflowing the textbox.
This is more visible in RTL builds as we right-align the URL bar value there.
MozReview-Commit-ID: IfLzQBhDXDt
--HG--
extra : rebase_source : 41f342bc9e5c02951599f7243d97151fd66c4de7
I see the following JavaScript warning in stdout when I run Firefox tests from the console.
JavaScript warning: resource://gre/modules/addons/XPIProvider.jsm, line 2970: String.localeCompare is deprecated; use String.prototype.localeCompare instead
MozReview-Commit-ID: ERiTd3rQ4Wc
--HG--
extra : rebase_source : bf7e8d28b652f17ae6c8ae140828acbd73bfc0e0
Fixes#16949.
- [x] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes do not require tests because it's just a refactor
Source-Repo: https://github.com/servo/servo
Source-Revision: 594479fe15f810d148e2352951affe905b1e13ff
--HG--
extra : subtree_source : https%3A//hg.mozilla.org/projects/converted-servo-linear
extra : subtree_revision : 10616530535dc13f95f990416e72d58eee85ff2d