Commit Graph

6110 Commits

Author SHA1 Message Date
Andreea Pavel
069d6deacf Bug 1560378 - disabled test_bug1364364.html on android r=gbrown CLOSED TREE
Differential Revision: https://phabricator.services.mozilla.com//D44303

--HG--
extra : histedit_source : 563e357b1fdd765ca14d5726d6d10c1c0077265a
2019-08-31 21:40:18 +03:00
Kris Maglione
7f551385df Bug 1576407: Follow-up: Move IsDiscarded check to BrowsingContext::GetOpener. r=nika
MANUAL PUSH: Follow-up.
2019-08-30 13:42:27 -07:00
Kris Maglione
5e1358a830 Bug 1576407: Follow-up: Ignore window.opener get/set when BrowsingContext is discarded. r=nika
MANUAL PUSH: Can't update patch on phabricator.

Differential Revision: https://phabricator.services.mozilla.com/D43833

--HG--
extra : source : 3b0498064cbf45527e8cef8ef030d0bc4687b21c
extra : amend_source : 637670b8e71d7628a01280dc49fb9cf23344e18b
2019-08-28 21:50:44 +00:00
Kris Maglione
3230783043 Bug 1576407: Part 4 - Stop storing opener redundantly on nsGlobalWindowOuter. r=farre
Differential Revision: https://phabricator.services.mozilla.com/D43693

--HG--
extra : source : 19bf40067233299dbe4870e3a5698ae67027dcd9
2019-08-28 18:07:45 +00:00
Kris Maglione
99bc492498 Bug 1576407: Part 3 - Return window opener as a WindowProxyHolder or BrowsingContext. r=farre
Also adds a legacy `GetSameProcessOpener()` method for callers which can only
deal with in-process windows and may need to be updated for Fission.

Differential Revision: https://phabricator.services.mozilla.com/D43692

--HG--
extra : source : 0ddbc451dcab431c2a1d934fa9baa5e8efc71545
2019-08-28 18:07:38 +00:00
Kris Maglione
2b62621fe8 Bug 1576407: Part 1 - Add HadOriginalOpener flag to BrowsingContext. r=farre
This is currently only available on the outer window, but needs to move to
BrowsingContext in order from us to remove redundant opener tracking from the
former.

Differential Revision: https://phabricator.services.mozilla.com/D43690

--HG--
extra : source : be7cbc61f825fdfc36b20d476bef710ef275ad73
2019-08-28 18:07:31 +00:00
Edgar Chen
bba22051b5 Bug 1577334 - Remove declaration of CanonicalBrowsingContext::NotifySetUserGestureActivationFromIPC; r=nika
The definition was removed in bug 1530550.

Differential Revision: https://phabricator.services.mozilla.com/D43832

--HG--
extra : moz-landing-system : lando
2019-08-29 15:53:09 +00:00
Mihai Alexandru Michis
a105826a55 Backed out 6 changesets (bug 1576407) for causing wpt and mochitest failures. CLOSED TREE
Backed out changeset 3b0498064cbf (bug 1576407)
Backed out changeset 38e4ea431445 (bug 1576407)
Backed out changeset 19bf40067233 (bug 1576407)
Backed out changeset 0ddbc451dcab (bug 1576407)
Backed out changeset 08d4c25fc670 (bug 1576407)
Backed out changeset be7cbc61f825 (bug 1576407)
2019-08-29 03:00:47 +03:00
Kris Maglione
49be7509f5 Bug 1576407: Part 4 - Stop storing opener redundantly on nsGlobalWindowOuter. r=farre
Differential Revision: https://phabricator.services.mozilla.com/D43693

--HG--
extra : moz-landing-system : lando
2019-08-28 18:07:45 +00:00
Kris Maglione
a0c3533e18 Bug 1576407: Part 3 - Return window opener as a WindowProxyHolder or BrowsingContext. r=farre
Also adds a legacy `GetSameProcessOpener()` method for callers which can only
deal with in-process windows and may need to be updated for Fission.

Differential Revision: https://phabricator.services.mozilla.com/D43692

--HG--
extra : moz-landing-system : lando
2019-08-28 18:07:38 +00:00
Kris Maglione
a75b009eab Bug 1576407: Part 1 - Add HadOriginalOpener flag to BrowsingContext. r=farre
This is currently only available on the outer window, but needs to move to
BrowsingContext in order from us to remove redundant opener tracking from the
former.

Differential Revision: https://phabricator.services.mozilla.com/D43690

--HG--
extra : moz-landing-system : lando
2019-08-28 18:07:31 +00:00
Mihai Alexandru Michis
42b26a5a1a Merge inbound to mozilla-central. a=merge
--HG--
rename : dom/base/Location.cpp => dom/base/LocationBase.cpp
rename : dom/base/Location.h => dom/base/LocationBase.h
2019-08-27 01:31:58 +03:00
Andreea Pavel
fb6a57f9b8 Backed out 2 changesets (bug 1532994) for build bustages on a CLOSED TREE
Backed out changeset 5b1337b35e30 (bug 1532994)
Backed out changeset c906462eddc7 (bug 1532994)
2019-08-26 21:22:39 +03:00
Kris Maglione
4814bcf9a6 Bug 1532994: Part 2 - Support SetHref() and Replace() in RemoteLocationProxy. r=farre
Differential Revision: https://phabricator.services.mozilla.com/D43328

--HG--
extra : moz-landing-system : lando
2019-08-26 11:45:16 +00:00
Kris Maglione
773bf2d878 Bug 1532994: Part 1 - Move cross-origin-accessible Location logic to base class. r=farre
Differential Revision: https://phabricator.services.mozilla.com/D43327

--HG--
rename : dom/base/Location.cpp => dom/base/LocationBase.cpp
rename : dom/base/Location.h => dom/base/LocationBase.h
extra : moz-landing-system : lando
2019-08-26 10:40:08 +00:00
Kris Maglione
06bd2c54e9 Bug 1532994: Part 2 - Support SetHref() and Replace() in RemoteLocationProxy. r=farre
MANUAL PUSH: Can't update phabricator patch.

Differential Revision: https://phabricator.services.mozilla.com/D43328

--HG--
extra : rebase_source : 66fc9a03238921b6f72b0e6a88b1ee7769edc940
extra : histedit_source : 57e8defe42bc9954bab8455bfd1f15aa2f45d01f
2019-08-23 16:20:37 -07:00
Kris Maglione
c1bf7db314 Bug 1532994: Part 1 - Move cross-origin-accessible Location logic to base class. r=farre
Differential Revision: https://phabricator.services.mozilla.com/D43327

--HG--
rename : dom/base/Location.cpp => dom/base/LocationBase.cpp
rename : dom/base/Location.h => dom/base/LocationBase.h
extra : rebase_source : 304f876fac6b06d4707b78b93acc1fdfe3d9a4f2
2019-08-23 15:26:51 -07:00
Andreas Farre
6e42fb98f8 Bug 1563619 - Handle subframe crashes in BrowserParent::ActorDestroy. r=mconley,kmag,peterv
Differential Revision: https://phabricator.services.mozilla.com/D40937

--HG--
extra : moz-landing-system : lando
2019-08-26 13:08:32 +00:00
Cosmin Sabou
8c496b626d Bug 1546159 - Disable test_bug565388.xul for frequent failures. r=jmaher
Differential Revision: https://phabricator.services.mozilla.com/D43027

--HG--
extra : moz-landing-system : lando
2019-08-22 08:21:20 +00:00
Thomas Nguyen
bbd6f51a56 Bug 1528697 - Add cenum referrerpolicy in ReferrerInfo.idl r=ckerschb,Gijs
Differential Revision: https://phabricator.services.mozilla.com/D41955

--HG--
extra : moz-landing-system : lando
2019-08-21 13:28:23 +00:00
Thomas Nguyen
32ab8293ff Bug 1528697 - Expose ReferrerPolicy.webidl and use referrerpolicy enum r=smaug
ReferrerPolicy gets tossed back and forth as a uint32_t and
ReferrerPolicy enum in header file. Expose ReferrerPolicyValues from
webidl file and use consistently in native code.

Differential Revision: https://phabricator.services.mozilla.com/D41954

--HG--
extra : moz-landing-system : lando
2019-08-21 13:24:45 +00:00
Sylvestre Ledru
7759b614e2 Bug 1575249 - Ride along: remove +x permissions on source files r=Ehsan
Depends on D42672

Differential Revision: https://phabricator.services.mozilla.com/D42673

--HG--
extra : moz-landing-system : lando
2019-08-21 09:57:03 +00:00
Kris Maglione
214bc6110f Bug 1561705: Part 3 - Get rid of remaining ChromePowers.js consumers. r=mccr8
Differential Revision: https://phabricator.services.mozilla.com/D42180

--HG--
extra : rebase_source : 4992d6ec6926e9052c2760765fb2f1a07f4a46d0
extra : source : cbe56a9fce5238a9bfdd0a2d226dc319380490b4
2019-08-14 16:18:32 -07:00
Abdoulaye O. Ly
5ce82c5c12 Bug 1562990 - [Fission] Make AudioPlayback work with JSWindowActor. r=NeilDeakin,farre
Differential Revision: https://phabricator.services.mozilla.com/D36648

--HG--
extra : moz-landing-system : lando
2019-08-19 21:17:21 +00:00
Kashav Madan
8f75b0e335 Bug 1574319 - Remove ContentParent::Recv* skip-if annotations, r=kmag
Differential Revision: https://phabricator.services.mozilla.com/D42374

--HG--
extra : moz-landing-system : lando
2019-08-16 21:00:17 +00:00
Kershaw Chang
fb1430c0de Bug 1553267 - Cancel the redirect load if docshell is being destroyed r=mayhemer
When the docshell is being destroyed, HttpChannelChild::CompleteRedirectSetup is not called and HttpChannelChild::mListener is not also assigned. In this case, I think we should cancel the channel and propagate the error to the http channel parent.

Differential Revision: https://phabricator.services.mozilla.com/D41515

--HG--
extra : moz-landing-system : lando
2019-08-16 11:41:04 +00:00
Kris Maglione
be074abe5e Bug 1571502: Follow-up: Remove straggler shutdown leak annotations after rebase.
Differential Revision: https://phabricator.services.mozilla.com/D42021

--HG--
extra : moz-landing-system : lando
2019-08-15 18:06:18 +00:00
Brindusan Cristian
d674f265db Merge mozilla-central to autoland. a=merge CLOSED TREE
--HG--
rename : layout/reftests/mathml/dir-6-ref.html => testing/web-platform/tests/mathml/presentation-markup/direction/direction-006-ref.html
2019-08-15 12:50:36 +03:00
Brindusan Cristian
3a61fb322f Merge inbound to mozilla-central. a=merge 2019-08-15 12:45:55 +03:00
Kris Maglione
78a2d6337e Bug 1573254: Follow-up: Fix bustage after rebase. r=bustage,test-only
--HG--
extra : amend_source : 619aee07e3bc085f866d6624bf352c06a80320e2
2019-08-14 14:36:03 -07:00
Jon Coppeard
c772fc6cda Bug 1573844 - Remove external references to js::Class r=mccr8
Depends on D41985

Differential Revision: https://phabricator.services.mozilla.com/D41986

--HG--
extra : moz-landing-system : lando
2019-08-14 17:15:15 +00:00
Boris Zbarsky
21c9b13b76 Bug 1422334. replaceState should update all the URI state for the entry being replaced. r=smaug
If we don't update the resultPrincipalURI, then things that examine it
(e.g. Location APIs and the URL bar) will show the wrong (pre-replaceState) URL.
I believe there is no effective difference between setting the result principal
URI to null and setting it to aNewURI here: the ultimate consumer of it is
NS_GetFinalChannelURI, which will fall back to the originalURI if it's null, and
in this case the originalURI is aNewURI.

Differential Revision: https://phabricator.services.mozilla.com/D41788

--HG--
extra : moz-landing-system : lando
2019-08-14 19:29:49 +00:00
Olli Pettay
1373bb3701 Bug 1563587, Make history.back/forward/go asynchronous, r=farre
The main part of the change is the change to ChildSHistory - make it possible to have Go() to be called asynchronously
and also let one to cancel pending history navigations. History object (window.history) can then use either the sync or
async Go(), depending on the dom.window.history.async pref.

LoadDelegate, which is used by GeckoView, needs special handling, since
it spins event loop nestedly. With session history loads and same-document loads we can just
bypass it.
To deal with same-document case, MaybeHandleSameDocumentNavigation is split to IsSameDocumentNavigation,
which collects relevant information about the request and returns true if same-document navigation should happen,
and then later HandleSameDocumentNavigation uses that information to trigger the navigation.
SameDocumentNavigationState is used to pass the information around.

referrer-policy-test-case.sub.js is buggy causing tests to pass only on Firefox with sync history API.

nested-context-navigations-iframe.html.ini is added because of https://bugzilla.mozilla.org/show_bug.cgi?id=1572932

Differential Revision: https://phabricator.services.mozilla.com/D41199

--HG--
extra : moz-landing-system : lando
2019-08-14 06:38:47 +00:00
Coroiu Cristina
3745a4a26f Backed out 2 changesets (bug 1563587) for junit failures on a CLOSED TREE
Backed out changeset d07f2e7d6f5e (bug 1563587)
Backed out changeset a365d3c43261 (bug 1563587)
2019-08-13 22:33:52 +03:00
Andrew McCreight
c706a636a8 Bug 1559489, part 4 - Remote-to-local window transplanting. r=tcampbell,bzbarsky
This patch cleans up remote outer window proxies when we navigate back
into the process.

It adds a flag to mDanglingRemoteOuterProxies that is set in between
BrowsingContext::SetDocShell(), where we can tell that the browsing
context is going from being remote to being local, to
nsGlobalWindowOuter::SetNewDocument(), where the local outer window
proxy is actually created. Once the outer window is created, the
remote window proxies can be cleaned up in
CleanUpDanglingRemoteOuterWindowProxies().

The clean up is done by a process that is similar to object
transplanting, except that instead of looking in the cross-compartment
wrapper table for each compartment to find objects to be turned into
CCWs to the new object, it looks in the remote proxy map for each
compartment. SpiderMonkey doesn't know about the proxy maps, so this
has to be done by a new callback object CompartmentTransplantCallback.

Now that this cleanup is being done, it shouldn't be possible to wrap
a remote outer window proxy when the browsing context is local, so
MaybeWrapWindowProxy() can be simplified. I had to drop the assert
here that the browsing context has a window proxy because during clean
up we call wrap on a local outer window proxy before the BC gets the
window proxy set on it. I had the assert because my original plan was
to implicitly fix remote proxies during wrapping, but that is no
longer necessary.

Differential Revision: https://phabricator.services.mozilla.com/D38343

--HG--
extra : moz-landing-system : lando
2019-08-13 19:09:59 +00:00
Olli Pettay
924f5f1a55 Bug 1563587, Make history.back/forward/go asynchronous, r=farre
The main part of the change is the change to ChildSHistory - make it possible to have Go() to be called asynchronously
and also let one to cancel pending history navigations. History object (window.history) can then use either the sync or
async Go(), depending on the dom.window.history.async pref.

LoadDelegate, which is used by GeckoView, needs special handling, since
it spins event loop nestedly. With session history loads and same-document loads we can just
bypass it.
To deal with same-document case, MaybeHandleSameDocumentNavigation is split to IsSameDocumentNavigation,
which collects relevant information about the request and returns true if same-document navigation should happen,
and then later HandleSameDocumentNavigation uses that information to trigger the navigation.
SameDocumentNavigationState is used to pass the information around.

referrer-policy-test-case.sub.js is buggy causing tests to pass only on Firefox with sync history API.

nested-context-navigations-iframe.html.ini is added because of https://bugzilla.mozilla.org/show_bug.cgi?id=1572932

Differential Revision: https://phabricator.services.mozilla.com/D41199

--HG--
extra : moz-landing-system : lando
2019-08-13 17:11:35 +00:00
Sylvestre Ledru
645f2d5773 Bug 1519636 - Reformat recent changes to the Google coding style r=Ehsan
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D41559

--HG--
extra : moz-landing-system : lando
2019-08-13 07:15:25 +00:00
Cosmin Sabou
c317b7ed38 Merge mozilla-central to mozilla-inbound. CLOSED TREE 2019-08-10 01:17:41 +03:00
Cosmin Sabou
23d4f0c224 Bug 1570953 - Add || to the syntax. a=test-only CLOSED TREE
--HG--
extra : rebase_source : 5ac0edb58dea4702323aa19147523f518af4f2a5
extra : amend_source : be0e79da775f634e9745d09dd93c49109f33a6d2
2019-08-10 00:09:22 +03:00
Cosmin Sabou
7cf0c18d49 Bug 1570953 - Reposition the skip-if clause because of gecko decision task bustage. r=bustage-fix 2019-08-09 22:40:24 +03:00
Cosmin Sabou
dd8bc45f10 Bug 1570953 - Reposition the skip-if clause on fission on kmag request. a=test-only
--HG--
extra : rebase_source : dd0e2a7868bfefcc38c4461a923cc578e1185708
2019-08-09 22:22:39 +03:00
alwu
0f5563b84f Bug 1565689 - part5 : control media from chrome process. r=farre,baku
This patch implements how to use MediaController to control corresponding media in content processes.

Differential Revision: https://phabricator.services.mozilla.com/D38145

--HG--
extra : moz-landing-system : lando
2019-08-07 01:46:03 +00:00
Daniel Varga
0e2158bb36 Bug 1571063 - Removing comment lines, part of 8017e6d5cbb6. Relanded with the correct bug number on a CLOSED TREE DONTBUILD
--HG--
extra : source : 5a02186b16a593aac36cd6474dff7cb580716cfb
extra : histedit_source : 565bf23781e84ad7ee704e2be3f3b2c49d719f05
2019-08-09 03:00:17 +03:00
Daniel Varga
25fe006248 Backed out changeset 5a02186b16a5 (bug 1511177) for wrong bug number. On a CLOSED TREE DONTBUILD
--HG--
extra : rebase_source : 174c5893441eb670cdfaa4367a7bd99b620e3bd0
2019-08-09 03:10:13 +03:00
Daniel Varga
b349cb59f1 Bug 1511177 - Removing comment lines. On a CLOSED TREE DONTBUILD
--HG--
extra : rebase_source : f89ddc0183c3fa68df653275d6f645a12206bd79
2019-08-09 03:00:17 +03:00
Daniel Varga
3814a2adf5 Backed out changeset 14e11bbc0dd3 (bug 1571063) for geckoview-web-platform-tests failures at /html/cross-origin/anonymous.tentative.html
--HG--
extra : rebase_source : 48306727547384d7260d0fc59274de7806732785
2019-08-09 00:02:18 +03:00
Junior Hsu
895ebba7c1 Bug 1543068 - P1.5 Remove InheritedEmbedderPolicy r=nika
I believe it's the right thing to do.
COEP inherits opener's COEP and is overwritten if COEP by response is stricter.
i.e., we no more need the inherited one.

Differential Revision: https://phabricator.services.mozilla.com/D40143

--HG--
extra : moz-landing-system : lando
2019-08-08 18:56:30 +00:00
Junior Hsu
2247427ef2 Bug 1543068 - P1 Substitute Cross-Origin header with COEP r=nika
(a) Substitute Cross-Origin header with COEP
(b) Forcing cors in FetchDriver is removed since COEP doesn't need to IIUC

Differential Revision: https://phabricator.services.mozilla.com/D38788

--HG--
extra : moz-landing-system : lando
2019-08-08 18:38:03 +00:00
Nika Layzell
143941e2d4 Bug 1523638 - Part 9: Use provided 'WindowGlobalChild' actors to create the initial about:blank document, r=kmag
Differential Revision: https://phabricator.services.mozilla.com/D37656

--HG--
extra : moz-landing-system : lando
2019-08-08 16:07:12 +00:00
Nika Layzell
15f2abf9e1 Bug 1523638 - Part 8: Remove unused arguments from nsDocShell::Embed(), r=bzbarsky
Differential Revision: https://phabricator.services.mozilla.com/D37655

--HG--
extra : moz-landing-system : lando
2019-08-08 16:07:10 +00:00