This refactors the mozharness configs such that a specific flavor of an overall suite can still use the old
DesktopUnittestOutputParser. This is necessary because mochitest-jetpack still does not use the structured
logger.
MozReview-Commit-ID: E8EpSLH4xt2
--HG--
extra : rebase_source : be18e3e4aa1f693b563a82b90fdda2f93201d9ee
This uplifts the upstream change 22557d466eceb6ff6ba70ae30d2dcd87648cde0b from
https://github.com/kinetiknz/cubeb as a standalone patch suitable for uplift.
The major change to the Iterator is due to the fact that the end of a
LinkedList is represented by nullptr.
Also delete the type conversion functions which are no longer needed.
MozReview-Commit-ID: 2lYtFW9pSon
--HG--
extra : rebase_source : c822e937679e57a9f2d3ab034e5147fc3aea9e8e
In our coding rules, variable names of nsresult should be rv. Indeed, when you see |rv| in the code, you must assume that its type if nsresult.
However, a lot of code under editor/ uses |res| for the variables of nsresult. Let's replace |res| with |rv|.
And this patch improves following points:
1. When |rv| is set in both |if| and |else| block and they are check outside of them, this moves the check into each |if| and |else| block because even if the failure is notified with warning, you cannot see which case was performed and failed. This change makes it clear.
2. When |return rv;| returns non-error code because |rv| is checked with NS_ENSURE_SUCCESS() immediately before, setting replacing it with |return NS_OK;| is clearer.
3. Move declaration of |nsresult rv| into smaller scope as far as possible. This prevents setting rv to unexpected value and easier to check its value at reading the code.
MozReview-Commit-ID: 9MAqj7sFey3
--HG--
extra : rebase_source : 0fd316b851ea616b3a95d8c1afc111ff55e11993
NS_FOUND_TARGET is now declared as an error code. However, making it as a success code makes the code simpler.
First, this patch renames it to NS_SUCCESS_EDITOR_FOUND_TARGET because it's usual naming rule (according to the other declarations).
Next, FindTargetNode() should return it when a nest call of itself returns NS_SUCCESS_EDITOR_FOUND_TARGET as it does now.
Finally, removing the code overwriting NS_FOUND_TARGET with NS_OK from HTMLEditor::CreateDOMFragmentFromPaste() since it doesn't cause hitting NS_ENSURE_SUCCESS() and the variable, rv, will be overwritten with other method's result.
MozReview-Commit-ID: 6GgZptrXXQa
--HG--
extra : rebase_source : 79418fc83fb087e559221f895c59c7c334980456
This removes the ability for ClearKey to instantiate persistent-license
sessions using the EME APIs.
MozReview-Commit-ID: KXyuRNMJKIZ
--HG--
extra : rebase_source : fd5c66f8929dfdb28ceb5cb82a75181a9168ca81
Note StateObject::HandleResumeVideoDecoding() doesn't check mIsReaderSuspended
because if the reader is suspended, MDSM must be in the dormant state and won't
resume video decoding.
MozReview-Commit-ID: FY4rdFmk56C
--HG--
extra : rebase_source : 000837b59944ad3f2efb12a367d7cc12c7a6deab
extra : source : a11fd583471f8a8e496a3152bb73bc17c6345a63
If MDSM becomes invisible while decoding metadata, VisibilityChanged() will return early
because mInfo is still nothing. And MDSM will never suspend video decoding if there are no
visibility changes later. We should delay the check util the timer goes off and MDSM
needs to switch on the blank decoder.
MozReview-Commit-ID: Ids9MncpC2l
--HG--
extra : rebase_source : 77ab76b09bae3dec3447950a8f0a7fcffff565b7
extra : source : 4527aed6f0b4f3ac11981df4517adacf9a43b4a2
Ehsan Akhgari
Jan de Mooij
Gijs Kruitbosch
JW Wang
Luke Wagner
Bill McCloskey
Benjamin Smedberg
Paul Yang
ffxbld
Andi-Bogdan Postelnicu
Yoshi Huang
Matthew Gregan
Mason Chang
Phil Ringnalda
Kilik Kuo
Carsten "Tomcat" Book
Cameron McCormack
Mats Palmgren
Xue Fuqiao
Ting-Yu Lin
Masayuki Nakano
Chris Pearce