Граф коммитов

12 Коммитов

Автор SHA1 Сообщение Дата
Andrea Marchesini f8cf3b6565 Bug 1290021 - Implement a prototype version of Houdini "Worklets Level 1" spec - part 3 - Console API in worklet, r=smaug 2016-11-06 09:54:52 +01:00
Andrea Marchesini 8ccb3ca885 Bug 1301032 - Remove a warning when Console.cpp is compiled, r=smaug 2016-09-07 18:08:23 +02:00
Nicholas Nethercote 34dcc7b852 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");

--HG--
extra : rebase_source : 58788245021096efa8372a9dc1d597a611d45611
2016-09-02 17:12:24 +10:00
Andrea Marchesini b3d116ff7e Bug 1286487 - WorkerProxyToMainThreadRunnable must keep alive workers using WorkerHolder - part 2, r=me 2016-08-16 09:35:21 +02:00
Andrew McCreight c054f8ec15 Bug 1292289, part 3 - Remove dependency of Console on xpcprivate.h. r=mrbkap
This file only uses functions on nsIXPConnect, so it can get it from
nsContentUtils and stop including xpcprivate.h, which eliminates
another file from being rebuilt when xpcprivate.h gets touched.

MozReview-Commit-ID: AT5Khu0vkyf

--HG--
extra : rebase_source : fa40afb7df9afe1dede714c111947e4e85aee34e
2016-08-04 11:35:13 -07:00
Boris Zbarsky 13a6409e22 Bug 1280818 part 2. Change console to capture the first non-self-hosted subsumed frame when it just needs a caller location, instead of capturing the first frame and then possibly failing to get useful location info out of it because it's self-hosted. r=baku 2016-07-21 23:41:02 -04:00
Andrea Marchesini 43868a9da7 Bug 1282366 - Improve WorkerHolder use in Runnables, r=khuey 2016-07-04 08:19:10 +02:00
Andrea Marchesini aac2306f2c Bug 1269154 - Get rid of WorkerFeature: WorkerHolder, r=khuey
--HG--
rename : dom/cache/Feature.cpp => dom/cache/CacheWorkerHolder.cpp
rename : dom/cache/Feature.h => dom/cache/CacheWorkerHolder.h
rename : dom/workers/WorkerFeature.h => dom/workers/WorkerHolder.h
2016-06-23 10:53:14 +02:00
Carsten "Tomcat" Book 47aeb86e2c Backed out changeset 1c5d78c7ba43 (bug 1269154) for bustage on a CLOSED TREE
--HG--
rename : dom/cache/CacheWorkerHolder.cpp => dom/cache/Feature.cpp
rename : dom/cache/CacheWorkerHolder.h => dom/cache/Feature.h
rename : dom/workers/WorkerHolder.h => dom/workers/WorkerFeature.h
extra : rebase_source : 49f9e9ce0500ac441fe97878cf9308804926544f
2016-06-23 10:13:54 +02:00
Andrea Marchesini 45087a7970 Bug 1269154 - Get rid of WorkerFeature: WorkerHolder, r=khuey
--HG--
rename : dom/cache/Feature.cpp => dom/cache/CacheWorkerHolder.cpp
rename : dom/cache/Feature.h => dom/cache/CacheWorkerHolder.h
rename : dom/workers/WorkerFeature.h => dom/workers/WorkerHolder.h
2016-06-22 17:24:35 +02:00
Andrea Marchesini 64734bf74c Bug 1278838 - Remove separate worker binding for Performance API, r=smaug
--HG--
rename : dom/performance/nsPerformance.cpp => dom/performance/Performance.cpp
rename : dom/performance/nsPerformance.h => dom/performance/Performance.h
rename : dom/workers/Performance.cpp => dom/performance/PerformanceWorker.cpp
rename : dom/workers/Performance.h => dom/performance/PerformanceWorker.h
2016-06-09 19:04:42 +02:00
Andrea Marchesini 501b078b36 Bug 1278799 - Moving Console API into dom/console, r=smaug
--HG--
rename : dom/base/Console.cpp => dom/console/Console.cpp
rename : dom/base/Console.h => dom/console/Console.h
rename : dom/base/ConsoleAPI.manifest => dom/console/ConsoleAPI.manifest
rename : dom/base/ConsoleAPIStorage.js => dom/console/ConsoleAPIStorage.js
rename : dom/base/ConsoleReportCollector.cpp => dom/console/ConsoleReportCollector.cpp
rename : dom/base/ConsoleReportCollector.h => dom/console/ConsoleReportCollector.h
rename : dom/base/nsIConsoleAPIStorage.idl => dom/console/nsIConsoleAPIStorage.idl
rename : dom/base/nsIConsoleReportCollector.h => dom/console/nsIConsoleReportCollector.h
rename : dom/base/test/test_bug659625.html => dom/console/tests/test_bug659625.html
rename : dom/base/test/test_bug978522.html => dom/console/tests/test_bug978522.html
rename : dom/base/test/test_bug979109.html => dom/console/tests/test_bug979109.html
rename : dom/base/test/test_bug989665.html => dom/console/tests/test_bug989665.html
rename : dom/base/test/test_console.xul => dom/console/tests/test_console.xul
rename : dom/base/test/test_consoleEmptyStack.html => dom/console/tests/test_consoleEmptyStack.html
rename : dom/base/test/test_console_binding.html => dom/console/tests/test_console_binding.html
rename : dom/base/test/test_console_proto.html => dom/console/tests/test_console_proto.html
2016-06-08 14:46:50 +02:00