Commit Graph

2559 Commits

Author SHA1 Message Date
Boris Zbarsky
c23223ed6b Bug 1250975. Stop passing a JSContext argument to WorkerRunnable::PreDispatch and its overrides. r=khuey 2016-02-25 16:05:39 -05:00
Boris Zbarsky
7c0e305a4c Bug 1250963 part 2. Remove the unused JSContext argument of WorkerPrivate::ModifyBusyCount. r=khuey 2016-02-25 16:05:39 -05:00
Boris Zbarsky
bb9a728cc9 Bug 1250963 part 1. Change NotifyRunnable::Dispatch to not require a JSContext. r=khuey
The only reason NotifyRunnable::Dispatch needs a JSContext is so that it can call
ModifyBusyCount in Pre/PostDispatch.  The only reason that needs a JSContext is
to call Cancel(), which only needs it to call Notify(), which only needs it to
call NotifyPrivate, which only needs it to dispatch a NotifyRunnable.
2016-02-25 16:05:39 -05:00
Ehsan Akhgari
2b0a5105b9 Bug 1250985 - Part 2: Prevent copy constructing a Request object with navigate mode if a RequestInit member is present; r=bzbarsky 2016-02-25 19:49:06 -05:00
Wes Kocher
f6b5a82260 Backed out 2 changesets (bug 1250963) for build bustage CLOSED TREE
Backed out changeset 257324c2ae17 (bug 1250963)
Backed out changeset 0e868ee89abc (bug 1250963)
2016-02-25 13:41:42 -08:00
Wes Kocher
051775630e Backed out changeset bb404647f14c (bug 1250975) for build bustage CLOSED TREE 2016-02-25 13:41:09 -08:00
Boris Zbarsky
47b0d8b88d Bug 1250975. Stop passing a JSContext argument to WorkerRunnable::PreDispatch and its overrides. r=khuey 2016-02-25 16:05:39 -05:00
Boris Zbarsky
0d11c02079 Bug 1250963 part 2. Remove the unused JSContext argument of WorkerPrivate::ModifyBusyCount. r=khuey 2016-02-25 16:05:39 -05:00
Boris Zbarsky
c2eea14fae Bug 1250963 part 1. Change NotifyRunnable::Dispatch to not require a JSContext. r=khuey
The only reason NotifyRunnable::Dispatch needs a JSContext is so that it can call
ModifyBusyCount in Pre/PostDispatch.  The only reason that needs a JSContext is
to call Cancel(), which only needs it to call Notify(), which only needs it to
call NotifyPrivate, which only needs it to dispatch a NotifyRunnable.
2016-02-25 16:05:39 -05:00
dimi
60178489c9 Bug 1237992 - service worker activate should be executed after install onstatechange events are fired. r=bkelly 2016-02-25 09:03:28 +08:00
Joel Maher
0999b8f250 Bug 1250630 - remove PBackgroundTest and ifdef ENABLE_TEST blocks. r=khuey
MozReview-Commit-ID: 8LKHyjKWrkd
2016-02-23 11:55:50 -08:00
Daniel Holbert
123c5edeab backout c42ff6847631 (bug 1250572) for causing aborts during startup 2016-02-24 11:54:35 -08:00
Andrea Marchesini
8b3b17e9c9 Bug 1250572 - Force a parent object in MessagePort/Channel and in StructuredCloneHolder, r=smaug 2016-02-24 20:04:37 +01:00
Boris Zbarsky
d5c4414107 Bug 1249673. Muted errors should be turned into NetworkError DOMExceptions when returning from importScripts on workers, instead of becoming NS_ERROR_FAILURE. r=baku 2016-02-24 10:38:31 -05:00
Boris Zbarsky
00986fabb4 Bug 1249652 part 3. Simplify way we handle canceling when ScriptLoaderRunnable::RunInternal fails by canceling things with its actual failure code, so we don't have to guess which failed loads are actual failures and which are just canceled via this mechanism. r=baku,khuey
There is a bit of subtlety here with NS_BINDING_ABORTED.  Before these changes,
we would land in ReportLoadError, not do anything with NS_BINDING_ABORTED, and
just return.  If called from WorkerPrivate::Constructor we'd then go ahead and
throw it on the ErrorResult, but I'm pretty sure we never ended up with
NS_BINDING_ABORTED there.  If called from ScriptExecutorRunnable::WorkerRun, we
would proceed on to ScriptExecutorRunnable::PostRun and hence
ShutdownScriptLoader where we would throw on the ErrorResult but NOT on the
JSContext.  Then we would unwind to our consumer and if that consumer was a
toplevel script load we would suppress the exception on the ErrorResult.
Otherwise we'd go ahead and throw the exception we ended up with to the caller.

The upshot is that we used to not fire error events on a worker whose main
script load was canceled with NS_BINDING_ABORTED.  So we try to preserve that
behavior explicitly for toplevel scripts.
2016-02-24 10:38:31 -05:00
Boris Zbarsky
6f1d1a266b Bug 1249652 part 2. ScriptExecutorRunnable::WorkerRun should immediately move JS exceptions to its ErrorResult instead of allowing them to linger on the JSContext. r=baku,khuey 2016-02-24 10:38:31 -05:00
Boris Zbarsky
7e66099590 Bug 1250291 part 3. Stop pretending to report exceptions in WorkerDebuggerRunnable::PostDispatch. r=khuey 2016-02-23 15:02:43 -05:00
Boris Zbarsky
307a76a0c0 Bug 1250291 part 2. Stop pretending to report exceptions in MainThreadWorkerSyncRunnable::PostDispatch. r=khuey 2016-02-23 15:02:42 -05:00
Boris Zbarsky
aeed64f72a Bug 1250291 part 1. Stop pretending to report exceptions in MainThreadStopSyncLoopRunnable::PostDispatch. r=khuey 2016-02-23 15:02:41 -05:00
Carsten "Tomcat" Book
c0d742a388 Merge mozilla-central to mozilla-inbound 2016-02-23 16:56:04 +01:00
Carsten "Tomcat" Book
c7adc23ebe merge fx-team to mozilla-central a=merge 2016-02-23 16:54:50 +01:00
Ben Kelly
aa16a9169f Bug 1247970 - Remove principal spec from service worker registrar file. r=baku 2016-02-15 14:41:00 +01:00
Eddy Bruel
7cb62ff777 Bug 1119490 - Expose the URL constructor to WorkerDebuggerGlobalScope;r=khuey 2016-02-22 10:41:09 +01:00
Ben Kelly
5e00e2ec52 Bug 1249351 part 2. Clean up test_importscript.html and add multiple-url importScript() case. r=bz 2016-02-22 17:13:42 -05:00
Boris Zbarsky
eb4e7528de Bug 1249351 part 1. When doing importScripts of multiple scripts in a service worker, make sure to track the cache streams per-loadinfo, instead of trying to make them all wait on the same stream. r=bkelly 2016-02-22 17:13:38 -05:00
Andrew McCreight
fed281470b Bug 1247679, part 3 - Replace NS_IMPL_CYCLE_COLLECTION_TRACE_JSVAL_MEMBER_CALLBACK with JS_MEMBER. r=smaug 2016-02-22 10:11:02 -08:00
Ben Kelly
197dcdce6f Bug 1242482 P4 Don't call SendUnregister() a second time when SW registration is finally removed. r=baku 2016-02-15 13:40:00 +01:00
Ben Kelly
2b21e4a6a3 Bug 1242482 P3 Don't send unregister messages when triggered from a PropagateUnregister(). r=baku 2016-02-20 07:03:00 +01:00
Ben Kelly
c9cae84370 Bug 1242482 P2 Don't SendUnregister() if registration is already removed. r=baku 2016-02-12 14:21:00 +01:00
Alexandre Poirot
8a7753f958 Bug 1242482 - Propagate Service worker unregistration to the parent process. r=bkelly 2016-01-25 07:49:00 +01:00
Xidorn Quan
84038e63d4 Bug 1248851 part 1 - Explicitly mark some release() calls result-unused. r=Waldo
MozReview-Commit-ID: Cgubemm1Et3
2016-02-20 11:06:25 +08:00
Boris Zbarsky
91cf3278af Bug 1249102. Make overrides of WorkerRunnable::PostRun a bit more consistent. r=khuey
Specifically we make the following changes:

1)  Remove WorkerSameThreadRunnable::PostRun, because it does exactly the same
things as WorkerRunnable::PostRun.

2)  Always treat ModifyBusyCountFromWorker as infallible in terms of throwing
JS exceptions.

3)  Change ExtendableFunctionalEventWorkerRunnable::PostRun to properly call
its superclass PostRun so we will correctly decrement the busy count our
PreDispatch incremented.

4)  Document why some overrides of PreDispatch/PostDispatch are needed and
don't call into the superclass
2016-02-18 18:02:51 -05:00
Boris Zbarsky
1ac0bf2b79 Bug 1248737. Improve documentation for WorkerRunnable and associated classes. r=khuey 2016-02-18 18:02:51 -05:00
Ben Kelly
ce114f7675 Bug 1246319 P3 Fix service worker registry value update. r=bz 2016-02-18 14:48:49 -08:00
Boris Zbarsky
2ad08651e7 Bug 1248719. Fix things so that taking ownership of error reporting on an AutoJSAPI on a worker is OK even if that AutoJSAPI was initialized without an explicit global. r=bholley 2016-02-17 22:56:36 -05:00
Kyle Huey
e971692182 Bug 1247117: De-namespace much of IndexedDB. r=baku 2016-02-16 13:46:08 -08:00
Aidin Gharibnavaz
ea7185b2c6 Bug 1235830 - Renaming GetInternalNSEvent to WidgetEvent, in nsIDOMEvent r=smaug
MozReview-Commit-ID: Hm8Owq17KXn
2016-02-12 19:10:07 +03:30
ISHIKAWA, Chiaki
72d38a6835 Bug 1248252 - Improper outdated octal constant syntax in M-C tree. Use '0o' prefix. r=dao
Be warned. Do not attemp to change the .js "test" source code in ./js
They are meant to check

 - the outdated 0666 octal constant is still parsed correctly,
 - the outdated 0666 octal constant raises syntax error flag
   in strict mode, etc.

So leave them alone.
2016-02-15 08:57:00 +01:00
Ben Kelly
93d9de0318 Bug 1246319 P2 Verify entries are deduped from the ServiceWorkerRegistrar. r=baku 2016-02-15 11:00:33 -08:00
Ben Kelly
c65d3e0f10 Bug 1246319 P1 Dedupe service worker registrar entries. r=baku 2016-02-15 11:00:33 -08:00
Andrea Marchesini
b532e955dc Bug 1247122 - Propagate the NS_ERROR_OUT_OF_MEMORY from CycleCollectedJSRuntime CTOR, r=khuey 2016-02-14 13:30:25 +00:00
Ben Kelly
fdbde537db Bug 1247623 Cancel intercepted channel in more service worker failure cases. r=jdm 2016-02-11 14:37:45 -08:00
Andrea Marchesini
9970b95cc1 Bug 1246784 - Expose Console to the WorkerDebuggerGlobalScope, r=khuey 2016-02-11 17:41:17 +00:00
Ben Kelly
9991d2d343 Bug 1247436 Ensure service worker registration is persisted if its resurrected from a pending uninstall. r=baku 2016-02-11 06:23:11 -08:00
Aidin Gharibnavaz
e084f12c9a Bug 1164581 - Adding an overload for NS_ProxyRelease that accepts already_AddRefed, and removing all the others. r=bobbyholley 2016-02-10 08:23:00 +01:00
Eddy Bruel
6142f35ea1 Bug 1212333 - WorkerDebuggerManager should live on the main thread;r=khuey 2016-02-10 10:45:02 +01:00
Ben Kelly
d9246c22d5 Bug 1238954 P1 Purge potentially controlled document entries when evicting service worker registrations. r=ehsan 2016-02-09 19:33:40 -08:00
Carsten "Tomcat" Book
ab60132384 Backed out changeset c18e29c1b369 (bug 1164581) for cpp unit tests test failures 2016-02-08 11:17:38 +01:00
Aidin Gharibnavaz
238dcd01fc Bug 1164581 - Adding an overload for NS_ProxyRelease that accepts already_AddRefed, and removing all the others. r=bobbyholley 2016-02-07 10:56:00 +01:00
Carsten "Tomcat" Book
be0e35fb43 Backed out changeset a6bc2efcdf26 (bug 1212333) for bustage 2016-02-05 16:09:28 +01:00