[UITest] Fix UITest CSP

RESOLVED WORKSFORME

Status

RESOLVED WORKSFORME
6 years ago
5 years ago

People

(Reporter: alive, Assigned: tjao)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=alive][mentor-lang=zh][miou])

Many parts in UITest app is broken due to CSP now.
(Assignee)

Updated

5 years ago
Depends on: 939021
(Assignee)

Updated

5 years ago
No longer depends on: 939021
Whiteboard: [good first bug][mentor=alive][mentor-lang=zh]
Whiteboard: [good first bug][mentor=alive][mentor-lang=zh] → [good first bug][mentor=alive][mentor-lang=zh][miou]
Is it still a valid issue since all UITest cases work now
Flags: needinfo?(alive)
If you really confirm everything works just close.
Flags: needinfo?(alive)

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WORKSFORME
I'm pretty sure window.open doesn't work.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
@alive, Any hint to fix window.open issue? @tom could help it
Assignee: alive → tjao
(Assignee)

Comment 5

5 years ago
OK, I'll take some time to survey this problem.
(Assignee)

Comment 6

5 years ago
I found that window.close() is broken, I'll fix it later. In other hand, window.open(url, name, 'attention') will open a popup window (Is this attention screen?) with transparent background. Though it's kind weird, the code in UITest seems to be OK.
Flags: needinfo?(alive)
It seems working now.
Flags: needinfo?(alive)
thanks Tom for studying and Alive for clarify
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.