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

8088 Коммитов

Автор SHA1 Сообщение Дата
Chris AtLee d1fb386dae Bug 1555560: Add support for black formatting with mozlint; r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D33125
2020-07-08 22:08:08 +00:00
Dorel Luca 584ee0d8de Backed out 5 changesets (bug 1647881, bug 1645521, bug 1645324) for BC failures in urlbar/tests/browser/browser_autocomplete_a11y_label.js. CLOSED TREE
Backed out changeset 65f908569875 (bug 1645324)
Backed out changeset 3080a3cacd0a (bug 1645521)
Backed out changeset d32236f070bf (bug 1645521)
Backed out changeset 579362aab769 (bug 1645521)
Backed out changeset 2b306b83c0d1 (bug 1647881)
2020-07-09 07:57:42 +03:00
Harry Twyford 8b234ca0d2 Bug 1645521 - Part 3 - Port unifiedcomplete tests. r=adw
Differential Revision: https://phabricator.services.mozilla.com/D80294
2020-07-09 02:13:11 +00:00
Nick Alexander aa96e0fc28 Bug 1651130 - Part 2: Add a new `tools/update-programs` build project. r=froydnj
Here "update programs" refers to the various standalone binaries we
produce in order to update Firefox.  There's not strong conceptual
coherence between them; rather, it's the pieces that the
Install/Update team generally work on.

To use this build project, prepare a minimal mozconfig with
`--enable-project=tools/update-programs`.  Depending on the mozconfig
options and host and target OS, some of the following will be built:

1. the maintenance service;
2. the updater binary;
3. the Windows Default Browser Agent;
4. the Background Update Agent.

Differential Revision: https://phabricator.services.mozilla.com/D82644
2020-07-08 23:03:21 +00:00
Mihai Alexandru Michis 5f385df474 Backed out 2 changesets (bug 1555560) for causing multiple lint failures.
CLOSED TREE

Backed out changeset bd9460ac6e48 (bug 1555560)
Backed out changeset f2896032ced6 (bug 1555560)
2020-07-09 01:04:18 +03:00
Chris AtLee 7f8f324f06 Bug 1555560: Add support for black formatting with mozlint; r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D33125
2020-07-08 19:35:49 +00:00
Nika Layzell 22a65a237e Bug 1650163 - Part 1: Switch native remoteType values to nsCString, r=farre,geckoview-reviewers,agi
Differential Revision: https://phabricator.services.mozilla.com/D82104
2020-07-08 20:15:59 +00:00
Mihai Alexandru Michis 1ba2a3f6f6 Backed out 3 changesets (bug 1650163) for causing bustages in nsContentSecurityManager.cpp
CLOSED TREE

Backed out changeset 51d7c644a1e6 (bug 1650163)
Backed out changeset 3d2b6908447a (bug 1650163)
Backed out changeset 79141707d47b (bug 1650163)
2020-07-08 21:18:44 +03:00
Nika Layzell c850a94434 Bug 1650163 - Part 1: Switch native remoteType values to nsCString, r=farre,geckoview-reviewers,agi
Differential Revision: https://phabricator.services.mozilla.com/D82104
2020-07-08 14:54:48 +00:00
emilfs dafec58d06 Bug 1643293 - added new job in doc to render documentation at review phase r=sylvestre,ahal
Differential Revision: https://phabricator.services.mozilla.com/D80428
2020-07-08 13:53:45 +00:00
Dimi Lee 8f56bdc39d Bug 1437626 - P4. Test requests sent in a service worker is classified r=baku
This patches does the following:
1. Test fetch in service worker should be classified.
2. Test requests should not be classified when it is whitelisted.
3. Test requests should not be classified when it is a first-party.

Depends on D80184

Differential Revision: https://phabricator.services.mozilla.com/D80185
2020-07-08 12:27:14 +00:00
Gregory Mierzwinski 8d10f86507 Bug 1650871 - Add all browsertime desktop tests. r=perftest-reviewers,AlexandruIonescu
This patch adds all desktop pageload tests to browsertime. It adds taskcluster tasks for these but doesn't enable them in production. They can only be run when --full is used.

Differential Revision: https://phabricator.services.mozilla.com/D82432
2020-07-08 07:48:16 +00:00
Manish Giri c61d108740 Bug 1650306 - mozlint/pylint - Enable 'no-else-return' rule r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D82507
2020-07-08 09:24:04 +00:00
Julian Descottes a99306e1b0 Bug 1650453 - Remove rejected words violations in devtools source code files r=nchevobbe
Depends on D82568

Differential Revision: https://phabricator.services.mozilla.com/D82212
2020-07-07 17:10:13 +00:00
Narcis Beleuzu 8359f16846 Backed out 7 changesets (bug 1650163, bug 1649477) for bustages on JSActor.cpp . CLOSED TREE
Backed out changeset 4a21afb65254 (bug 1650163)
Backed out changeset c41753a56f5a (bug 1650163)
Backed out changeset 5fb444c35764 (bug 1650163)
Backed out changeset 830aa93d2b0c (bug 1649477)
Backed out changeset eca6e9dce450 (bug 1649477)
Backed out changeset 5b217aa88289 (bug 1649477)
Backed out changeset 8959d02b840f (bug 1649477)
2020-07-08 04:09:27 +03:00
Nika Layzell df351180c3 Bug 1650163 - Part 1: Switch native remoteType values to nsCString, r=farre
Differential Revision: https://phabricator.services.mozilla.com/D82104
2020-07-06 20:30:58 +00:00
Nika Layzell a3579f6f46 Bug 1458043 - Stop converting promises to nsISupports in xpconnect, r=kmag
Differential Revision: https://phabricator.services.mozilla.com/D82616
2020-07-07 22:20:01 +00:00
Nick Alexander 40947e8d1d Bug 1651156 - Use HOST_BIN_SUFFIX in clang-format lint. r=ahal
This allows cross-compiling to Windows targets.

Differential Revision: https://phabricator.services.mozilla.com/D82582
2020-07-07 18:26:54 +00:00
Dave Townsend c551d1b151 Bug 1642398: Add a lint rule to warn about multiple calls to document.l10n.formatValue. r=Standard8,preferences-reviewers,ntim
Differential Revision: https://phabricator.services.mozilla.com/D77900
2020-07-02 18:51:44 +00:00
Tom Prince 9ef67b9fdd No bug: Make `.taskcluster.yml` yamllint clean; r=aki
Differential Revision: https://phabricator.services.mozilla.com/D82243
2020-07-03 23:30:59 +00:00
Mihai Alexandru Michis 3a33785d38 Backed out changeset 9b15a67a50c3 (bug 1650834) as requested by Sylvestre.
CLOSED TREE
2020-07-07 11:49:36 +03:00
Sylvestre Ledru 9c734f1175 Bug 1650834 - Make tools/lint code black compatible r=ahal
Differential Revision: https://phabricator.services.mozilla.com/D82401
2020-07-07 08:09:31 +00:00
Gerald Squelart 6447251c76 Bug 1640325 - FileIO markers show the type of file if known - r=canaltinova
Platform-specific observations will be able to specify a file type.
They will appear in distinct rows in the profiler.firefox.com Marker Chart.
The default type is "File", which is shown in markers as "FileIO" like before.

Differential Revision: https://phabricator.services.mozilla.com/D80399
2020-07-06 23:42:55 +00:00
Razvan Maries 65bc3a986b Backed out changeset 4911fdc92832 (bug 1650834) for Docker Images bustages. CLOSED TREE 2020-07-06 22:30:17 +03:00
Sylvestre Ledru 16145a8de0 Bug 1650834 - Make tools/lint code black compatible r=ahal
Differential Revision: https://phabricator.services.mozilla.com/D82401
2020-07-06 18:11:28 +00:00
Andrew Halberstadt 3a5cc7c7bc Bug 1649194 - [tryselect] Add a documentation page for |mach try auto|, r=marco
Differential Revision: https://phabricator.services.mozilla.com/D82241
2020-07-03 21:52:19 +00:00
Julian Descottes f0547c8165 Bug 1650155 - Update exclude list in rejected-words.yml after renaming whitelistRejectionsGlobally to allowMatchingRejectionsGlobally r=Gijs
Depends on D82061

Differential Revision: https://phabricator.services.mozilla.com/D82062
2020-07-03 16:57:38 +00:00
Geoff Brown e17ca5f1b5 Bug 1585726 - Run mozlint on macosx64 and win64 too; r=ahal
Differential Revision: https://phabricator.services.mozilla.com/D82200
2020-07-03 16:07:33 +00:00
Andrew Halberstadt 47d424ae5d Bug 1648586 - [lint] Add missing 'colorama' to pylint_requirements.txt when on Windows, r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D82126
2020-07-03 12:16:50 +00:00
Julian Descottes 84942bd174 Bug 1618059 - Add test_extension_storage_actor_upgrade.js to the rejected-words exclude list. a=lint-fix CLOSED TREE
test_extension_storage_actor_upgrade.js was originally created from test_extension_storage_actor.js
Both tests use the PromiseTestUtils.whitelistRejectionsGlobally API which fails the rejected-words linter.

While we wait for the API to be renamed, I am adding the new test to the exclude list.
2020-07-02 18:37:35 +02:00
Kershaw Chang e467f29f69 Bug 1650068 - Enable profiler tests r=JuniorHsu
Differential Revision: https://phabricator.services.mozilla.com/D82019
2020-07-02 15:25:44 +00:00
Mark Banner 9751545bb2 Bug 1618465 - Update references to pdf.js for the move to toolkit. r=bdahl
Differential Revision: https://phabricator.services.mozilla.com/D72533
2020-07-02 07:14:45 +00:00
Jim Porter cdcaab85f1 Bug 1649670 - Fix "control reaches end of non-void function" in ProfilerMarkerPayload.cpp; r=gerald
Differential Revision: https://phabricator.services.mozilla.com/D81884
2020-07-02 00:08:56 +00:00
Gerald Squelart 56736f31c3 Bug 1648507 - Distinguish pausing sampling only from pausing the whole profiler - r=canaltinova,perftest-reviewers,geckoview-reviewers,agi
The profiler can be "paused", which stops sampling, and since bug 1578329 stops markers as well.

Some test suites use pausing between tests (to better differentiate the tests, to keep the profiler ready to run, and to lower the amount of recorded data). But this causes problems with some tracing markers, as their matching ends have not been recorded (e.g., an end marker is missing), which show up as very loooong markers.
To solve this, we need to be able to pause sampling only, but keep recording markers.
But we still need to be able to pause the whole profiler, in particular before capturing, to avoid recording anything around that time.

This big patch is mostly mechanical changes: Wherever there are "Pause" and "Unpause/Resume" profiler functions, we add matching "PauseSampling" and "UnpauseSampling/ResumeSampling" functions that only impact the periodic sampling loop; And existing "Pause/Unpause/Resume" imply pausing sampling as well.
Exceptions and extra work:
- nsIProfiler (the JS API) already had `Pause/ResumeSampling()`, which misleadingly paused everything! Now they do the right thing, and we have `Pause/Resume()` as well.
- All tests using `Pause/ResumeSampling()` now use `Pause/Resume()`, except for Talos tests that only pause sampling between tests; Added some extra `Pause()` calls to pause everything before capturing profiles.
- GeckoJavaSampler doesn't handle pausing/resuming everything, this should be done in a follow-up bug.
- Sampling-only pauses are not streamed into JSON. If needed, we should follow-up, with potential work on the front-end to deal with these.

Differential Revision: https://phabricator.services.mozilla.com/D81492
2020-07-02 01:36:27 +00:00
Jean-Yves Avenard b2cf09ec3e Bug 1634846 - P2. Make ipc's MessageChannel works with TaskQueue, r=nika
We no longer rely of having a message loop for the worker thread.

Differential Revision: https://phabricator.services.mozilla.com/D80655
2020-07-02 00:26:41 +00:00
chocos 1e019373f9 Bug 1648959 - Done internal linking in docs in tools Directory DONTBUILD.r=championshuttler
Differential Revision: https://phabricator.services.mozilla.com/D81465
2020-07-01 19:35:26 +00:00
Simon Giesecke 9364b353d4 Bug 1648010 - Remove NS_NAMED_LITERAL_CSTRING and NS_NAMED_LITERAL_STRING macros. r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D80631
2020-07-01 08:42:31 +00:00
Simon Giesecke cd8b8939b9 Bug 1648010 - Replace uses of NS_LITERAL_STRING/NS_LITERAL_CSTRING macros by _ns literals. r=geckoview-reviewers,jgilbert,agi,hsivonen,froydnj
Differential Revision: https://phabricator.services.mozilla.com/D80860
2020-07-01 08:29:29 +00:00
Dorel Luca 675da38265 Backed out 4 changesets (bug 1648858) for Browser-chrome failures on content/test/static/browser_all_files_referenced.js. CLOSED TREE
Backed out changeset 2a107eabe00c (bug 1648858)
Backed out changeset 1622eb497b04 (bug 1648858)
Backed out changeset f5770886187d (bug 1648858)
Backed out changeset 3a9ae16f63c9 (bug 1648858)
2020-07-01 12:27:26 +03:00
Matthew Noorenberghe b550937547 Bug 1648858 - Move d3.js from devtools/ to toolkit/. r=mossop
Differential Revision: https://phabricator.services.mozilla.com/D81437
2020-07-01 06:33:05 +00:00
Geoff Brown cb19f2147e Bug 1647362 - Improve py3 compat in fix_stacks.py; r=bc
Use universal_newlines as a convenient way of treating the process I/O as text,
so string read/writes continue to succeed under python 3.

Differential Revision: https://phabricator.services.mozilla.com/D81774
2020-06-30 22:17:38 +00:00
Jim Porter 632c7aa9aa Bug 1613496 - Add IPC profiler markers to track when messages are processed by the IO thread queues r=gerald,froydnj
This adds 3 new profiler markers for each IPC message:

 * One just before the first byte is sent over the IPC channel
 * One just after the last byte is sent over the IPC channel
 * One just after the last byte is received from the IPC channel

With the already-existing IPC markers (for when SendXXX and RecvXXX are
called), this allows us to calculate the following statistics:

 * Send thread latency
 * IPC send duration
 * IPC recv latency
 * Recv thread latency

For more information on how this is presented in the UI, see:
<https://github.com/firefox-devtools/profiler/pull/2535>.

Differential Revision: https://phabricator.services.mozilla.com/D70790
2020-06-30 18:39:42 +00:00
Myeongjun Go 20bcc8c1aa Bug 1633418 - [Perfdocs] Add the ability to include static content into perfdocs r=sparky,perftest-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D80031
2020-06-30 16:35:00 +00:00
Paul Adenot ee148dcec6 Bug 1649233 - Add a new ProfilerMarkerPayload that is just a time duration. r=gerald,canaltinova
Differential Revision: https://phabricator.services.mozilla.com/D81592
2020-06-30 14:25:09 +00:00
Gerald Squelart c1fe2ae51c Bug 1649056 - Pre-record some meta information before locking the profiler mutex - r=canaltinova
Some profile meta information can be gathered before the profiler mutex must be locked.
This reduces the main-thread locking when capturing.
Most importantly, it prevents deadlocking in case any of the data-gathering operation would itself rely on profiler-locking functions (e.g., starting a thread, which requires the lock to register the new thread with the profiler).

Differential Revision: https://phabricator.services.mozilla.com/D81491
2020-06-30 13:40:13 +00:00
Narcis Beleuzu 3de598d4a7 Backed out 2 changesets (bug 1649233) for bustages on ProfilerMarkerPayload.h . CLOSED TREE
Backed out changeset 399e54895a75 (bug 1649233)
Backed out changeset 3955edb371cf (bug 1649233)
2020-06-30 16:36:06 +03:00
Paul Adenot d9cb2a1db6 Bug 1649233 - Add a new ProfilerMarkerPayload that is just a time duration. r=gerald,canaltinova
Differential Revision: https://phabricator.services.mozilla.com/D81592
2020-06-30 12:10:00 +00:00
Gerald Squelart 3fbe4a7b38 Bug 1648972 - Fix profiler non-unified build - r=canaltinova
Mostly missing includes, and missing namespace qualifiers.

Differential Revision: https://phabricator.services.mozilla.com/D81451
2020-06-30 05:07:21 +00:00
Narcis Beleuzu 3700aab557 Backed out 7 changesets (bug 1634846, bug 1647628, bug 1649294, bug 1647112) for webgl-conf crashes. CLOSED TREE
Backed out changeset 4441d06e96c3 (bug 1647628)
Backed out changeset 4efaf32bc8f7 (bug 1647112)
Backed out changeset 2d24ad813039 (bug 1647112)
Backed out changeset fda262d73a13 (bug 1649294)
Backed out changeset 5863f9c5229f (bug 1634846)
Backed out changeset bca79526745d (bug 1634846)
Backed out changeset d539408a0048 (bug 1634846)
2020-06-30 09:50:00 +03:00
Narcis Beleuzu 51f0465656 Backed out 2 changesets (bug 1648972, bug 1649056) for assertion failure on ExtensionPolicyService.cpp . CLOSED TREE
Backed out changeset 6a8b446a95a0 (bug 1649056)
Backed out changeset 56b166e41900 (bug 1648972)
2020-06-30 08:00:31 +03:00