Closed Bug 1988719 Opened 4 months ago Closed 1 month ago

Preload scripts are not run for windows opened with window.open() (without a URL argument)

Categories

(Remote Protocol :: WebDriver BiDi, defect, P3)

defect

Tracking

(firefox148 fixed)

RESOLVED FIXED
148 Branch
Tracking Status
firefox148 --- fixed

People

(Reporter: hbenl, Assigned: vhilla)

References

Details

(Whiteboard: [webdriver:m18][webdriver:external])

See this comment.
Bug 1988202 may be related since it also shows a different behavior for window.open() vs. window.open("about:blank").

Preload scripts are also not run for dynamically added iframes, which is why this Playwright test is currently failing.

(In reply to Holger Benl [:hbenl] from comment #1)

Preload scripts are also not run for dynamically added iframes, which is why this Playwright test is currently failing.

I was wrong about this: the preload scripts are also run for dynamically added iframes, the Playwright test is failing for a different reason that I'm still investigating.

The severity field is not set for this bug.
:whimboo, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(hskupin)

Sasha, will take a look until next week's meeting.

Severity: -- → S3
Flags: needinfo?(hskupin) → needinfo?(aborovova)
Priority: -- → P3
Whiteboard: [webdriver:backlog][webdriver:triage]

The issue is really with document-element-inserted notification not being dispatched, but seems like bug 543435 fixes it and then preload scripts are executed (I tested it with the attached patches). But the timing issue from bug 1985997 is still present.

Depends on: sync-about-blank
Flags: needinfo?(aborovova)
Whiteboard: [webdriver:backlog][webdriver:triage] → [webdriver:backlog]

(In reply to Alexandra Borovova [:Sasha] from comment #5)

The issue is really with document-element-inserted notification not being dispatched, but seems like bug 543435 fixes it and then preload scripts are executed (I tested it with the attached patches). But the timing issue from bug 1985997 is still present.

Sasha, did you mean to add bug 1985997 to the dependency list for this bug or isn't it a hard blocker because it's under the see also section?

Flags: needinfo?(aborovova)

Sasha, did you mean to add bug 1985997 to the dependency list for this bug or isn't it a hard blocker because it's under the see also section?

bug 543435 was the only blocker. I've just retested it and can not reproduce it anymore. So I think we can consider it fixed.

Status: NEW → RESOLVED
Closed: 1 month ago
Flags: needinfo?(aborovova)
Resolution: --- → FIXED

Bug 543435 is going to be backed out from beta. So lets mark it fixed for 148 instead.

Assignee: nobody → vhilla
Target Milestone: --- → 148 Branch
Whiteboard: [webdriver:backlog] → [webdriver:m18][webdriver:external]
You need to log in before you can comment on or make changes to this bug.