Commit Graph

667 Commits

Author SHA1 Message Date
Ben Kelly
43459db366 Bug 1300658 P1 Expose a main thread TaskQueue WorkerPrivate. r=baku 2016-09-12 12:32:21 -07:00
Ben Kelly
449d185850 Backout rev 5c7368370ff9 to 980659720b86 (bug 1300118 and bug 1300658) for incorrect bug number in commit message. r=me 2016-09-12 12:29:17 -07:00
Ben Kelly
28c3944fe8 Bug 1300658 P2 Make WorkerRunnable's destined for main thread use the MainThreadTaskQueue. r=baku 2016-09-12 11:21:01 -07:00
Ben Kelly
c88a12768d Bug 1300658 P1 Expose a main thread TaskQueue WorkerPrivate. r=baku 2016-09-12 11:21:01 -07:00
Tom Tung
1f61d7f9c3 Bug 1187335 - P1 - Add a way to report error to all clients for ServiceWorker and SharedWorker. r=bkelly. 2016-05-23 14:56:46 +08:00
Michael Layzell
f2f13378b5 Bug 1018486 - Part 1: Changes in dom/, r=baku
MozReview-Commit-ID: 4tCUM4KRe81
2016-09-07 10:50:35 -04:00
Nicholas Nethercote
b5810a1eb4 Bug 1299384 - Use MOZ_MUST_USE with NS_warn_if_impl(). r=erahm.
This change avoids lots of false positives for Coverity's CHECKED_RETURN
warning, caused by NS_WARN_IF's current use in both statement-style and
expression-style.

In the case where the code within the NS_WARN_IF has side-effects, I made the
following change.

> NS_WARN_IF(NS_FAILED(FunctionWithSideEffects()));
> -->
> Unused << NS_WARN_IF(NS_FAILED(FunctionWithSideEffects()));

In the case where the code within the NS_WARN_IF lacks side-effects, I made the
following change.

> NS_WARN_IF(!condWithoutSideEffects);
> -->
> NS_WARNING_ASSERTION(condWithoutSideEffects, "msg");

This has two improvements.
- The condition is not evaluated in non-debug builds.
- The sense of the condition is inverted to the familiar "this condition should
  be true" sense used in assertions.

A common variation on the side-effect-free case is the following.

> nsresult rv = Fn();
> NS_WARN_IF_(NS_FAILED(rv));
> -->
> DebugOnly<nsresult rv> = Fn();
> NS_WARNING_ASSERTION(NS_SUCCEEDED(rv), "Fn failed");
2016-09-02 17:12:24 +10:00
Andrea Marchesini
51a746281b Bug 1297472 - Improve nsIScriptTimeoutHandler::GetHandlerText(), r=bz 2016-09-05 21:25:13 +02:00
Nicholas Nethercote
887efe04d5 Bug 1299727 - Rename NS_WARN_IF_FALSE as NS_WARNING_ASSERTION. r=erahm.
The new name makes the sense of the condition much clearer. E.g. compare:

  NS_WARN_IF_FALSE(!rv.Failed());

with:

  NS_WARNING_ASSERTION(!rv.Failed());

The new name also makes it clearer that it only has effect in debug builds,
because that's standard for assertions.
2016-09-01 15:01:16 +10:00
Nicholas Nethercote
d62d70da42 Bug 1299389 - Replace some raw pointers in nsThreadManager. r=froydnj.
nsThreadManager::get() can return a reference. This lets us remove some
redundant assertions.

nsThreadArray elements can be NotNull<>s.
2016-06-10 16:04:49 +10:00
Andrea Marchesini
5f8dfa9ba6 Bug 790919 - Don't dispatch close event in Workers, r=bkelly 2016-08-31 21:33:05 -07:00
Boris Zbarsky
69f901291a Bug 986459. The error event in a worker (firing at the worker's global) should have the original exception value in its .error property. r=baku 2016-08-29 12:30:51 -04:00
Nicholas Nethercote
1009bf0056 Bug 1297658 - Avoid unnecessary checking in memory reporters. r=erahm.
This patch removes checking of all the callback calls in memory reporter
CollectReport() functions, because it's not useful.

The patch also does some associated clean-up.

- Replaces some uses of nsIMemoryReporterCallback with the preferred
  nsIHandleReportCallback typedef.

- Replaces aCallback/aCb/aClosure with aHandleRepor/aData for CollectReports()
  parameter names, for consistency.

- Adds MOZ_MUST_USE/[must_use] in a few places in nsIMemoryReporter.idl.

- Uses the MOZ_COLLECT_REPORT macro in all suitable places.

Overall the patch reduces code size by ~300 lines and reduces the size of
libxul by about 37 KiB on my Linux64 builds.
2016-08-24 15:23:45 +10:00
Andrea Marchesini
f563f8e14f Bug 1296586 - Fixing some comments in WorkerPrivate, r=smaug 2016-08-21 08:41:34 +02:00
Ben Kelly
a111356a39 Bug 1293690 P1 Allow WorkerHolder::HoldWorker() callers to specify the status value they consider to be a failure. r=baku 2016-08-18 07:11:04 -07:00
Phil Ringnalda
6ef267a68f Backed out 10 changesets (bug 1290116, bug 1293690) for build bustage
CLOSED TREE

Backed out changeset bbc14d107fca (bug 1290116)
Backed out changeset a81b6ce5c25f (bug 1290116)
Backed out changeset e0b9c25b6125 (bug 1290116)
Backed out changeset 8e9a9ba04ce5 (bug 1290116)
Backed out changeset 2d5ad5e9ed20 (bug 1290116)
Backed out changeset eb7da0dbfc72 (bug 1290116)
Backed out changeset 6a4e8c92faae (bug 1290116)
Backed out changeset 0342d98afa10 (bug 1293690)
Backed out changeset d84b0768f0ce (bug 1293690)
Backed out changeset f9f62ade9508 (bug 1293690)
2016-08-17 21:16:43 -07:00
Ben Kelly
af00c54873 Bug 1293690 P1 Allow WorkerHolder::HoldWorker() callers to specify the status value they consider to be a failure. r=baku 2016-08-17 20:01:35 -07:00
Andrea Marchesini
c8a40161f0 Bug 1264053 - MessagePort should support transferable objects in multi-e10s, r=sfink, r=smaug, r=jorendorff, r=janv 2016-07-21 15:29:42 +02:00
Andrea Marchesini
8219258940 Bug 1288770 - Switch worker timeouts to using nsJSTimeoutHandler, r=smaug 2016-08-16 08:10:30 +02:00
Arthur Edelstein
621139730b Bug 1174386 - Make workers inherit the default Intl locale from the main thread, rather than using a bogus fallback value. r=jwalden, r=baku 2016-08-08 13:33:39 -07:00
Sebastian Hengst
31d3ea3143 Backed out changeset 2d5975fd02bd (bug 1288770) for asserting in ErrorResult.h when test test_errorPropagation.html runs. r=backout 2016-08-16 17:11:22 +02:00
Andrea Marchesini
1a1510ed53 Bug 1288770 - Switch worker timeouts to using nsJSTimeoutHandler, r=smaug 2016-08-16 08:10:30 +02:00
Boris Zbarsky
5e4ad72a49 Bug 1291364. Make sure workers unmark things gray before passing them to JS_CallFunctionValue when running timeouts. r=terrence 2016-08-02 11:31:02 -07:00
Eric Rahm
570d676a3c Bug 1290288 - Remove control runnable warning. r=khuey 2016-08-02 10:03:47 -07:00
Ben Kelly
4464e9d2fe Bug 1289658 Make ExtendableMessageEvent.waitUntil() hold the service worker alive. r=asuth 2016-07-27 10:32:12 -04:00
Jan de Mooij
a7e86841f6 Bug 1286795 part 2 - Change JS_RequestInterruptCallback to take JSContext instead of JSRuntime. r=luke 2016-07-23 19:52:20 +02:00
Andrea Marchesini
3e782b8fc6 Bug 1288736 - Add some missing rv.SuppressException(), r=smaug 2016-07-22 16:50:10 +02:00
Iris Hsiao
5dec23519d Backed out changeset ba6b75f29652 (bug 1288031) for web platform e10s test failure 2016-07-21 17:44:44 +08:00
Andrea Marchesini
e408b72b64 Bug 1288031 - CloseHandlerStarted() is called wrongly in WorkerPrivate, r=khuey 2016-07-21 09:18:32 +02:00
Alexandre Lissy
e4fd2dab03 Bug 1286530 - Clean AvailableIn, CheckAnyPermissions and CheckAllPermissions from WebIDL r=bz,fabrice
MozReview-Commit-ID: 6EQfBM09xUE
2016-04-21 15:48:59 +02:00
Boris Zbarsky
c3889c8b5d Bug 1279313 part 1. Simplify AutoEntryScript to not make callers pass in a JSContext. r=bholley 2016-07-07 20:08:25 -04:00
Jan de Mooij
7c3e2f8f9a Bug 1283855 part 28 - Make more GC APIs take JSContext instead of JSRuntime. r=terrence 2016-07-07 09:56:09 +02:00
Jan de Mooij
43c0eb9efc Bug 1283855 part 26 - Make more GC APIs take JSContext instead of JSRuntime. r=terrence,mccr8 2016-07-07 09:55:41 +02:00
Jan de Mooij
6941dba9aa Bug 1284808 - Rename RuntimeOptions to ContextOptions and move it to the context. r=luke,baku 2016-07-07 08:15:15 +02:00
Andrea Marchesini
169fe0974d Bug 1282992 - Improve sync event loop shutdown in workers, r=khuey 2016-07-06 08:36:54 +02:00
Jan de Mooij
22f78cde9e Bug 1283855 part 12 - Make some GC APIs take JSContext instead of JSRuntime. r=sfink 2016-07-05 14:35:21 +02:00
Jan de Mooij
1f565193bd Bug 1283855 part 7 - Make JS_SetGCZeal take JSContext instead of JSRuntime. r=sfink 2016-07-05 11:06:05 +02:00
Thomas Nguyen
8ad6fcfed0 Bug 1251378 - Propagate referrer policy to worker. r=ckerschb
MozReview-Commit-ID: 6BlEl54Dqtc
2016-07-05 11:47:13 +08:00
Andrea Marchesini
2127b52ff8 Bug 1283418 - In WorkerPrivate we often use !mSyncLoopStack.Length() instead mSyncLoopStack.IsEmpty(), r=smaug 2016-07-04 08:18:53 +02:00
Kyle Huey
ce16c8e90c Bug 1282603: Don't use 'workers': True for *WorkerGlobalScope. r=baku 2016-06-28 10:47:23 -07:00
Andrea Marchesini
611ada4ec2 Bug 1280067 - WorkerThreadModifyBusyCount should be the default value for WorkerRunnable, r=sicking 2016-06-28 19:28:13 +02:00
Andrea Marchesini
796bda032a Bug 1269154 - Get rid of WorkerFeature: WorkerHolder, r=khuey 2016-06-23 10:53:14 +02:00
Carsten "Tomcat" Book
ef49710e52 Backed out changeset 1c5d78c7ba43 (bug 1269154) for bustage on a CLOSED TREE 2016-06-23 10:13:54 +02:00
Andrea Marchesini
274f4e927f Bug 1269154 - Get rid of WorkerFeature: WorkerHolder, r=khuey 2016-06-22 17:24:35 +02:00
Andrea Marchesini
7dcf0fcf39 Bug 1278838 - Remove separate worker binding for Performance API, r=smaug 2016-06-09 19:04:42 +02:00
Boris Zbarsky
dd0c0086a9 Bug 1276317 part 1. Change JS_SetGCZeal to take a JSRuntime, not a JSContext. r=jandem 2016-06-02 10:34:38 -04:00
Xidorn Quan
4843f879c0 Bug 1276549 - Remove rvalue reference mark for already_AddRefed params of nsIEventTarget::Dispatch and related methods. r=froydnj
MozReview-Commit-ID: J5RAfGW3X7T
2016-06-01 10:04:54 +10:00
Jan de Mooij
edde813ac1 Bug 1274922 part 5 - Remove JSEXN_NONE and add JSEXN_WARN. r=jorendorff 2016-05-31 08:07:00 +02:00
Boris Zbarsky
5d75957ebc Bug 1276297. Remove the unused WorkerPrivateParent::ParentJSContext function. r=khuey 2016-05-27 13:30:33 -04:00
Boris Zbarsky
c5857caa43 Bug 1270783. If we end up erroring out of a worker main-script load before we get to creating a global for the worker, don't try to enter that global's compartment to throw an exception. Just squelch it instead. r=khuey 2016-05-13 20:09:50 -04:00