Open Bug 1552296 Opened 5 years ago Updated 2 years ago

Raptor incorrectly sets environment variables for mobile browsers except Fennec

Categories

(Testing :: Raptor, defect, P3)

Product:
Testing
Component:
Raptor
Version:
Version 3
Type:
defect

Tracking

(Not tracked)

People

(Reporter: whimboo, Unassigned)

Details

(In reply to Henrik Skupin (:whimboo) [⌚️UTC+2] from bug 1528236 comment #27)
The code which this revision tried to improve can be found here:
https://searchfox.org/mozilla-central/rev/94c6b5f06d2464f6780a52f32e917d25ddc30d6b/testing/raptor/raptor/raptor.py#897-918

Actually launch_application() doesn't support moz_env, but only launch_fennec(). And the latter moves those entries into the extra dictionary as root properties:

https://searchfox.org/mozilla-central/rev/94c6b5f06d2464f6780a52f32e917d25ddc30d6b/testing/mozbase/mozdevice/mozdevice/adb.py#3082-3086

Raptor actually puts those under extras["args"] which is basically wrong, because that is used for actual browser arguments.

Priority: -- → P2
Assignee: nobody → onegru
Priority: P2 → P1
Assignee: onegru → nobody
Priority: P1 → P2
Assignee: nobody → marian.raiciof
Priority: P2 → P1
Status: NEW → ASSIGNED
Assignee: marian.raiciof → nobody
Status: ASSIGNED → NEW
Priority: P1 → P2

This is more a P3 as a P2.

Priority: P2 → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.