9eeeb4f28ddc43ee8b15b13ccc2ceb21b1af7bbb: Correctly account for two-byte source text allocation when XPCJSSourceHook allocates it, to return a count of bytes, not of two-byte code units. No bug, rs=bz over IRC

Bug 1519952, don't use the boxobject screenX and screenY properties in the tooltip listener, instead get the screen coordinates from the frame directly, r=bz

41da7f72eb6eb912a702170e912db116195f6beb: Bug 1519952, add screenX and screenY properties to XULElement in order to replace places that currently get the screen coordinates of an element via the box object, r=bz

Bug 1435899 - Use nsStringStream for the data channel buffer. r=bz
A pipe is no longer used for the input stream, instead we use a string stream
which in most cases will be able to share the string data buffer rather than
copying it.

a5e00e5b4635a19e9fb4ed3566565504338384c6: Bug 1506163 - Do not span column-span:all element across all columns if it's under different block formatting context. r=bz

Bug 1506163 - Do not span column-span:all element across all columns if it's under different block formatting context. r=bz
multicol-span-all-004-ref.html is the same as multicol-span-all-004.html except
for the "column-span" value in h3.
Differential Revision: https://phabricator.services.mozilla.com/D12192

Bug 1196668 - Make images without alt generate a replaced box regardless of src. r=bz
Behavior-wise this only removes the HasAttr(src) check, and adds the IsEmpty()
check to the alt attribute value, since this function is only called for <img>
and <input>.
But it also cleans up a bit.
Differential Revision: https://phabricator.services.mozilla.com/D11194

Bug 1196668 - Make images without alt generate a replaced box regardless of src. r=bz
Behavior-wise this only removes the HasAttr(src) check, and adds the IsEmpty()
check to the alt attribute value, since this function is only called for <img>
and <input>.
But it also cleans up a bit.
Differential Revision: https://phabricator.services.mozilla.com/D11194

Bug 1196668 - Make images without alt generate a replaced box regardless of src. r=bz
Behavior-wise this only removes the HasAttr(src) check, and adds the IsEmpty()
check to the alt attribute value, since this function is only called for <img>
and <input>.
But it also cleans up a bit.
Differential Revision: https://phabricator.services.mozilla.com/D11194

Bug 1356412: Part 2 - Remove script loader path mangling from DOM code. r=bz
As in part 1, this no longer serves any purpose.
This patch also removes the otherwise unused WrappersEnabled() method, which
was similarly only useful before we had compartment-based security isolation.
Its functionality is now handled by compartment flags.
Differential Revision: https://phabricator.services.mozilla.com/D10796

bb6607c98119eda64ea5e431cafe93e683569f5f: Bug 1503602 - Use the Response's global to create the ReadableStream reader to consume the body, r=bz

Bug 1486698 - Update Fetch+Stream implementation to throw when the stream is disturbed or locked, r=bz
In this patch, I went through any place in DOM fetch code, where there are
ReadableStreams and update the locked, disturbed, readable checks.
Because we expose streams more often, we need an extra care in the use of
ErrorResult objects. JS streams can now throw exceptions and we need to handle
them.
This patch also fixes a bug in FileStreamReader::CloseAndRelease() which could
be called in case mReader creation fails.

Bug 1492648 - Move from nsDocShellLoadInfo to nsDocShellLoadState; r=bz, nika
Creates the nsDocShellLoadState object, which is basically
nsDocShellLoadInfo plus a few extra fields to make it usable as a
single argument to nsDocShell::LoadURI (and eventually
nsDocShell::InternalLoad).
Subframe history handling is a huge logic block in
nsDocShell::LoadURI, which is only used on history loads. This patch
also extracts the logic out into its own function to make the body of
LoadURI clearer.

ab03a5f493975fe3403bbcdbcab4d1075f1a9a42: Bug 1501354 - Fetch() should not block the loadgroup from completing, r=bz

Bug 1494745 part 4 - Make Loader::AddObserver void since it's infallible. r=bz
mObservers is a nsTObserverArray which is infallible, so no need to
check the return value from AppendElementUnlessExists
(which a later patch in this series will remove).

Bug 1494745 part 3 - Make nsRefreshDriver::AddTimerAdjustmentObserver void since it's infallible. r=bz
mTimerAdjustmentObservers is a nsTObserverArray which is infallible,
so no need to check the return value from AppendElement
(which a later patch in this series will remove).

Bug 1494745 part 2 - Make nsRefreshDriver::AddRefreshObserver void since it's infallible. r=bz
nsRefreshDriver::ObserverArray is a nsTObserverArray which is
infallible, so no need to check the return value from AppendElement
(which a later patch in this series will remove).

4ac8ee3c801972a1c47e8270dccc801612c3aa21: Bug 1494745 part 1 - Don't check the result from ns[Auto]TObserverArray::AppendElement since it's infallible. r=bz

Bug 1494745 part 1 - Don't check the result from ns[Auto]TObserverArray::AppendElement since it's infallible. r=bz
mChildList and mListenerInfoList are ns[Auto]TObserverArray which are
infallible, so no need to check the return value from AppendElement
(which a later patch in this series will remove).