Ben Hearsum
0220aa4c47
Bug 1874944: send distinct telemetry data for empty attribution strings r=nalexander
...
An empty attribution string is a different case from one that exists that cannot be read (they generally point towards different potential causes in our infrastructure). We should send them as distinct errors.
Differential Revision: https://phabricator.services.mozilla.com/D199558
2024-01-26 00:26:58 +00:00
Joel Maher
3662d84cd4
Bug 1873732 - remove outdated manifest conditions and use consistent conditions. r=aryx,necko-reviewers,jgilbert,settings-reviewers,credential-management-reviewers,devtools-reviewers,sessionstore-reviewers,dom-storage-reviewers,sgalich,kershaw,nchevobbe,janv,dao
...
Differential Revision: https://phabricator.services.mozilla.com/D198082
2024-01-25 00:27:05 +00:00
Stanca Serban
71d0d5cf96
Backed out changeset 54462970f797 (bug 1873732) for causing web platform tests failures.
2024-01-24 21:08:19 +02:00
Joel Maher
04ec786571
Bug 1873732 - remove outdated manifest conditions and use consistent conditions. r=aryx,necko-reviewers,jgilbert,settings-reviewers,credential-management-reviewers,devtools-reviewers,sessionstore-reviewers,dom-storage-reviewers,sgalich,kershaw,nchevobbe,janv,dao
...
Differential Revision: https://phabricator.services.mozilla.com/D198082
2024-01-24 16:35:54 +00:00
Ben Hearsum
95abbbe2b4
Bug 1874177: enable macOS attribution reporting by default r=nalexander
...
Differential Revision: https://phabricator.services.mozilla.com/D198294
2024-01-11 20:39:40 +00:00
Ben Hearsum
ec42f5ce45
Bug 1865845: stop caching mac attribution data in a separate file r=nalexander
...
This is mostly removing code and tests related to reading/writing the cache file on macOS and updating of tests. Aside from that, the most notable part is the change to `setAttributionString` to automatically prepend `__MOZCUSTOM__` when writing an attribution code. This is mostly done to make things simpler and cleaner in the majority of the tests, but seeing as `getAttributionString` is also aware of it, it seems like a generally nicer way to do this.
Differential Revision: https://phabricator.services.mozilla.com/D197204
2024-01-09 20:21:54 +00:00
Narcis Beleuzu
a68fbd0a88
Backed out 3 changesets (bug 1865845) for bc failures on browser_AttributionCode_telemetry.js . CLOSED TREE
...
Backed out changeset 4f799b89c628 (bug 1865845)
Backed out changeset e825fb03ff56 (bug 1865845)
Backed out changeset 9a8c396a22db (bug 1865845)
2024-01-09 21:26:48 +02:00
Ben Hearsum
b2ea441dc1
Bug 1865845: stop caching mac attribution data in a separate file r=nalexander
...
This is mostly removing code and tests related to reading/writing the cache file on macOS and updating of tests. Aside from that, the most notable part is the change to `setAttributionString` to automatically prepend `__MOZCUSTOM__` when writing an attribution code. This is mostly done to make things simpler and cleaner in the majority of the tests, but seeing as `getAttributionString` is also aware of it, it seems like a generally nicer way to do this.
Differential Revision: https://phabricator.services.mozilla.com/D197204
2024-01-09 15:38:04 +00:00
Michael Hughes
c3b8c6535a
Bug 1830725: Synchronous nsWindowsPackageManager::GetCampaignId blocks main thread for many seconds r=nrishel
...
Made GetCampaignId asynchronous to not block anything.
Also fix Bug 1872933 (broken build)
Differential Revision: https://phabricator.services.mozilla.com/D188520
2024-01-05 21:26:39 +00:00
Iulian Moraru
847ded45c7
Backed out changeset f7c78cadff5d (bug 1830725) for causing bc failures on browser_AttributionCode_telemetry.js.
2024-01-04 01:53:19 +02:00
Michael Hughes
22f7c8b815
Bug 1830725: Synchronous nsWindowsPackageManager::GetCampaignId blocks main thread for many seconds r=nrishel
...
Made GetCampaignId asynchronous to not block anything.
Differential Revision: https://phabricator.services.mozilla.com/D188520
2024-01-03 21:50:04 +00:00
Norisz Fay
8c94855abe
Backed out changeset 416268cc8a36 (bug 1830725) for causing bc failures on browser_AttributionCode_telemetry.js
2023-11-29 00:56:57 +02:00
Michael Hughes
7c539a4288
Bug 1830725: Synchronous nsWindowsPackageManager::GetCampaignId blocks main thread for many seconds r=bytesized,nrishel
...
Made GetCampaignId asynchronous to not block anything.
Differential Revision: https://phabricator.services.mozilla.com/D188520
2023-11-28 18:24:34 +00:00
Joel Maher
33fa84a382
Bug 1859869 - batch 12 - browser/* xpcshell .ini manifests to .toml. r=aryx,extension-reviewers,credential-management-reviewers,search-reviewers,sessionstore-reviewers,places-reviewers,issammani,Standard8,robwu,sclements
...
Differential Revision: https://phabricator.services.mozilla.com/D193996
2023-11-21 17:35:23 +00:00
Ben Hearsum
16a5f7c21b
Bug 1858545: put macOS attribution reporting behind a Nimbus flag r=nalexander
...
Differential Revision: https://phabricator.services.mozilla.com/D190986
2023-10-18 00:02:47 +00:00
Ben Hearsum
5156f0778e
Bug 1816998: Replace macOS quarantine database-based attribution with extended attribute attribution r=nalexander
...
With extended attribute attribution using exactly the same attribution strings as Windows (eg: no URLs), this turns out to be quite straightforward. Once we've pulled in the attribution string from the extended attribute, the existing parsing, validation, etc. just works.
The only wrinkle is that the extended attributes may have nul bytes or tabs that we need to strip away. (Tabs may be present because we use them to pad the attribution area when we prepare the DMG for attribution. Nul bytes may be present because we overwrite the entire attribution before updating the attribution data.)
Differential Revision: https://phabricator.services.mozilla.com/D189258
2023-10-05 23:30:57 +00:00
Robin Steuber
4bc61ea8a6
Bug 1815021 - Back out reading provenance data from AttributionCode r=nalexander
...
Backed out Bug 1815021's patch: https://hg.mozilla.org/mozilla-central/rev/9b8c31e9eb9f
Differential Revision: https://phabricator.services.mozilla.com/D189594
2023-10-03 23:40:25 +00:00
Robin Steuber
7ad18fece6
Bug 1855604 - Back out support for sending provenance data via telemetry r=nalexander
...
Backed out Bug 1815023's patch: https://hg.mozilla.org/mozilla-central/rev/fc0d63d3404b
Depends on D189592
Differential Revision: https://phabricator.services.mozilla.com/D189593
2023-10-03 23:40:25 +00:00
Robin Steuber
e194d9f28b
Bug 1855604 - Back out putting provenance attribution in installation.first_seen r=nalexander
...
Backs out Bug 1821189's patch: https://hg.mozilla.org/mozilla-central/rev/c4f8098f81bb
Differential Revision: https://phabricator.services.mozilla.com/D189591
2023-10-03 23:40:24 +00:00
Sylvestre Ledru
2e0e132821
Bug 1856530 - run file-whitespace on toml files r=linter-reviewers,Standard8
...
Differential Revision: https://phabricator.services.mozilla.com/D189896
2023-10-03 09:22:18 +00:00
Tom Marble
ad50861ef9
Bug 1853243 - convert .ini manifests to .toml: batch 5 browser/**/browser.ini r=jmaher,mossop,webcompat-reviewers,twisniewski,search-reviewers,Standard8
...
Differential Revision: https://phabricator.services.mozilla.com/D188427
2023-09-21 22:47:07 +00:00
Mark Banner
0666077666
Bug 1826062 - Automatic fixes for upgrading Prettier to 2.8.8. r=mossop,perftest-reviewers,webcompat-reviewers,geckoview-reviewers,denschub,devtools-reviewers,sparky,calu
...
Differential Revision: https://phabricator.services.mozilla.com/D177027
2023-05-20 12:26:53 +00:00
Mark Banner
91fde42f7b
Bug 1826062 - Automatic fixes for Prettier 2.0.5 upgrade. r=mossop,perftest-reviewers,webcompat-reviewers,geckoview-reviewers,denschub,devtools-reviewers,sparky,owlish
...
Differential Revision: https://phabricator.services.mozilla.com/D177025
2023-05-20 12:26:49 +00:00
anwar
1d2e079021
Bug 1829673 - Put all mochitest test manifest files under browser/components into alphabetical order. r=mconley,extension-reviewers,credential-management-reviewers,dimi,robwu
...
Differential Revision: https://phabricator.services.mozilla.com/D176459
2023-05-02 16:13:24 +00:00
CanadaHonk
de0de3489c
Bug 1801379 - Migrate Sinon.jsm to an ES module r=extension-reviewers,application-update-reviewers,pip-reviewers,credential-management-reviewers,fxview-reviewers,devtools-reviewers,Standard8,nchevobbe,sclements,dimi,mconley,bytesized,robwu
...
Migrated `testing/modules/Sinon.sys.mjs` to an ES module.
`testing` should now be 100% ESM 🎉
Differential Revision: https://phabricator.services.mozilla.com/D173643
2023-03-29 07:34:10 +00:00
William Durand
1b34df1849
Bug 1819997 - Allow dlsource in attribution data. r=mixedpuppy
...
Differential Revision: https://phabricator.services.mozilla.com/D173429
2023-03-28 19:41:36 +00:00
Robin Steuber
c7f37da43c
Bug 1821189 - Associate provenance attribution with installation.first_seen event r=nalexander
...
We can't add the provenance data to the `installation.first_seen` extra data because it is already at its maximum number of keys. So instead we will add the `installation.first_seen_prov_ext` event which will be sent at the same time as `installation.first_seen` and will contain provenance attribution data in its extras object.
Differential Revision: https://phabricator.services.mozilla.com/D172520
2023-03-14 18:08:16 +00:00
Robin Steuber
70d26915bc
Bug 1821406 - testProvenance calls should be awaited upon r=nalexander
...
Differential Revision: https://phabricator.services.mozilla.com/D172074
2023-03-09 17:28:37 +00:00
Robin Steuber
5eabcd66a1
Bug 1815023 - Add support for sending attribution provenance data via telemetry r=nalexander,chutten
...
Add a function that sets scalars to convey what kind of data is present in the :Zone.Identifier Alternate Data Stream. Also adds testing that the scalars get set correctly.
Does not actually call the function that sends the telemetry. This will be done in the next patch since we need to call it in some slightly different places than we usually would since we want to be sure it is included in the `new-profile` ping and in the ping with the `installation.first_seen` event.
Differential Revision: https://phabricator.services.mozilla.com/D171818
2023-03-08 22:25:07 +00:00
Robin Steuber
b8b2f03a55
Bug 1815021 - Update AttributionCode.jsm to read provenance data r=nalexander
...
Differential Revision: https://phabricator.services.mozilla.com/D171817
2023-03-08 22:25:06 +00:00
elephanteatsapple
0ad4eadcd9
Bug 1812141 - Convert JSMs in browser/components/attribution to ESMs. r=mconley,kpatenio,nalexander
...
Differential Revision: https://phabricator.services.mozilla.com/D167730
2023-02-15 21:40:30 +00:00
Marian-Vasile Laza
4f6b335406
Backed out changeset 0d0223fcdc02 (bug 1812141) for xpcshelll failures on test_ClientEnvironment.js. CLOSED TREE
2023-02-13 20:34:22 +02:00
elephanteatsapple
62845ac0d3
Bug 1812141 - Convert JSMs in browser/components/attribution to ESMs. r=mconley,kpatenio,nalexander
...
Differential Revision: https://phabricator.services.mozilla.com/D167730
2023-02-13 16:32:41 +00:00
Ben Hearsum
91a47e1267
Bug 1807924: make attribution tests work against MSIX builds r=nalexander
...
Two notable things here:
- Small refactor of AttributionCode.jsm to make it more testable (allowing `getCampaignId` to be easily stubbed out)
- An ugly skip of one of the "invalid" attribution code tests -- for reasons noted in the comments. Ultimately, this points to maybe a refactor or larger change being needed, but I don't think it's worth blocking getting these tests on.
Differential Revision: https://phabricator.services.mozilla.com/D166688
2023-01-16 19:32:05 +00:00
Ben Hearsum
01f485b050
Bug 1805919: disable tests that are currently failing against MSIX packages. r=jmaher,necko-reviewers,kershaw
...
Differential Revision: https://phabricator.services.mozilla.com/D165754
2023-01-11 14:45:03 +00:00
Mark Banner
df47c60747
Bug 1806359 - Convert telemetry imports to direct ES imports. r=necko-reviewers,application-update-reviewers,pip-reviewers,credential-management-reviewers,janerik,mconley,sgalich,bytesized,kershaw
...
Differential Revision: https://phabricator.services.mozilla.com/D165002
2022-12-23 12:37:07 +00:00
Ben Hearsum
c167bcedb4
Bug 1804894: Don't write postSigningData in MSIX builds r=nalexander
...
This is only done by the Return-to-AMO flow, which is not supported in MSIX anyways.
Also, disable the RTAMO test for MSIX builds while we're here.
Differential Revision: https://phabricator.services.mozilla.com/D164666
2022-12-15 14:30:09 +00:00
Iulian Moraru
929d19c3de
Backed out 11 changesets (bug 1804903, bug 1804902, bug 1804900, bug 1804899, bug 1712329, bug 1804898, bug 1804894, bug 1804904, bug 1804896, bug 1804905) for breaking gecko decision task. CLOSED TREE
...
Backed out changeset 9e7813271e6b (bug 1712329)
Backed out changeset 29aa1d032dd3 (bug 1804903)
Backed out changeset 22ef0b00cc99 (bug 1804905)
Backed out changeset 8fad832a574e (bug 1804904)
Backed out changeset 60f185a3a485 (bug 1804902)
Backed out changeset 8b69824de6e2 (bug 1804900)
Backed out changeset 140f75f3f919 (bug 1804899)
Backed out changeset f4da44b1a219 (bug 1804898)
Backed out changeset 7cf946ae8295 (bug 1804896)
Backed out changeset 07d62af0767f (bug 1804894)
Backed out changeset 1e3e02cc3b1f (bug 1712329)
2022-12-15 03:16:41 +02:00
Ben Hearsum
20e93dfd50
Bug 1804894: Don't write postSigningData in MSIX builds r=nalexander
...
This is only done by the Return-to-AMO flow, which is not supported in MSIX anyways.
Also, disable the RTAMO test for MSIX builds while we're here.
Differential Revision: https://phabricator.services.mozilla.com/D164666
2022-12-15 00:45:13 +00:00
Barret Rennie
d1a2566d45
Bug 1541508 - Use Services.env in browser/ r=mossop
...
Differential Revision: https://phabricator.services.mozilla.com/D160136
2022-11-25 19:09:06 +00:00
Cristian Tuns
c3ffdd5df6
Backed out 19 changesets (bug 1541508) for causing xpcshell failures on test_notHeadlessByDefault.js CLOSED TREE
...
Backed out changeset 08476fa2bc27 (bug 1541508)
Backed out changeset 0bf7514845db (bug 1541508)
Backed out changeset aa612a5e9ef7 (bug 1541508)
Backed out changeset 6bb9360473f7 (bug 1541508)
Backed out changeset b3d8e92f50c2 (bug 1541508)
Backed out changeset fa40dded133e (bug 1541508)
Backed out changeset 2e7db4aa8d4f (bug 1541508)
Backed out changeset 6098e2eb62ea (bug 1541508)
Backed out changeset 2c599ee639c4 (bug 1541508)
Backed out changeset 7d44f6e2644c (bug 1541508)
Backed out changeset c1279c3d674c (bug 1541508)
Backed out changeset 8bd08a62a590 (bug 1541508)
Backed out changeset 740010cb005c (bug 1541508)
Backed out changeset 0bfc7dd85c62 (bug 1541508)
Backed out changeset c4374a351356 (bug 1541508)
Backed out changeset 44ccfeca7364 (bug 1541508)
Backed out changeset e944e706a523 (bug 1541508)
Backed out changeset 2c59d66f43e4 (bug 1541508)
Backed out changeset a1896eacb6f1 (bug 1541508)
2022-11-01 22:38:52 -04:00
Barret Rennie
813aae1553
Bug 1541508 - Use Services.env in browser/ r=mossop
...
Differential Revision: https://phabricator.services.mozilla.com/D160136
2022-11-02 02:08:53 +00:00
Mark Banner
5407bdffa8
Bug 1792341 - Migrate more toolkit/modules consumers to use direct ES module import. r=Gijs,webdriver-reviewers,perftest-reviewers,necko-reviewers,geckoview-reviewers,preferences-reviewers,application-update-reviewers,pip-reviewers,credential-management-reviewers,sgalich,owlish,bytesized,AlexandruIonescu,whimboo,mconley,mixedpuppy
...
Mainly automated changes. Some manual ESLint fixes and whitespace cleanup.
Differential Revision: https://phabricator.services.mozilla.com/D158452
2022-10-18 11:21:26 +00:00
Mark Banner
19b7fb742e
Bug 1782659 - Enable ESlint rule no-unused-vars on the global scope for browser/ xpcshell-tests. r=mak
...
Differential Revision: https://phabricator.services.mozilla.com/D153433
2022-08-05 08:38:50 +00:00
Tooru Fujisawa
48c568867c
Bug 1667455 - Part 5: Stop importing Services.jsm from chrome-priv JS code, top-level single-line cases. r=kmag,webdriver-reviewers,perftest-reviewers,webcompat-reviewers,geckoview-reviewers,extension-reviewers,application-update-reviewers,pip-reviewers,twisniewski,m_kato,jdescottes,mconley,AlexandruIonescu,mossop
...
Differential Revision: https://phabricator.services.mozilla.com/D150894
2022-07-11 12:41:50 +00:00
Ben Hearsum
0b34b1b5bb
Bug 1766479: don't run Windows only attribution tests on Mac r=aryx
...
Differential Revision: https://phabricator.services.mozilla.com/D144729
2022-04-26 19:20:13 +00:00
Ben Hearsum
c3b0ed7d3d
Bug 1756209: send Windows Store Campaign ID as attribution code r=nalexander
...
Nearly all the complexity here is abstracted away in the previous patch, so this is quite straightforward other than the one oddity noted in the comments.
Differential Revision: https://phabricator.services.mozilla.com/D143112
2022-04-26 15:55:34 +00:00
Gijs Kruitbosch
edda40141b
Bug 1753696 - Automatic rewrite: replace add_task(setup with add_setup in browser mochitest, r=Standard8,webcompat-reviewers,twisniewski
...
Differential Revision: https://phabricator.services.mozilla.com/D141437
2022-03-30 16:16:42 +00:00
Norisz Fay
e1d7571cc8
Backed out 2 changesets (bug 1753696) for causing multiple dt and bc failures CLOSED TREE
...
Backed out changeset 67fd9edfbcea (bug 1753696)
Backed out changeset a9d957ea887b (bug 1753696)
2022-03-30 12:17:55 +03:00
Gijs Kruitbosch
1214ac636b
Bug 1753696 - Automatic rewrite: replace add_task(setup with add_setup in browser mochitest, r=Standard8,webcompat-reviewers,twisniewski
...
Differential Revision: https://phabricator.services.mozilla.com/D141437
2022-03-30 08:18:51 +00:00