Set window handle for "Switch To Window" also as "handle"

RESOLVED FIXED in Firefox 67

Status

enhancement
P3
normal
RESOLVED FIXED
4 months ago
2 months ago

People

(Reporter: whimboo, Assigned: kritisingh1.ks, Mentored)

Tracking

(Blocks 2 bugs, {good-first-bug})

unspecified
mozilla67
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox67 fixed)

Details

(Whiteboard: [lang=rs])

User Story

To get started follow the documentation at:

https://firefox-source-docs.mozilla.org/testing/marionette/marionette/NewContributors.html

https://firefox-source-docs.mozilla.org/testing/geckodriver/geckodriver/index.html

Attachments

(2 attachments, 1 obsolete attachment)

To get started on bug 1301073 we have to make geckodriver backward compatible by also setting the window handle as "handle" here:

https://searchfox.org/mozilla-central/rev/5053031ba7621fa8f63f42de4c204ab3561e4e59/testing/geckodriver/src/marionette.rs#1447

The "name" property should be left-in for now to stay compatible.

Priority: -- → P3

Hi Henrik I would like to work on it , I build the firefox locally on my laptop. Can you guide me how to go ahead with it?

Thanks

Flags: needinfo?(hskupin)

As comment 0 explains just add handle next to name as member of the data instance.

Flags: needinfo?(hskupin)
Assignee: nobody → shivams2799
(Assignee)

Comment 3

2 months ago

Hi I would like to take this up if its not active. Please let me know if I may. Thanks!

Kriti , I am working on it , had a rough last week sorry.

Thanks

Just to let you know we basically assign bugs to the first one who uploads a patch. So please upload one, or other might/will do it in the next days.

Assignee: shivams2799 → nobody
Status: ASSIGNED → NEW
Assignee: nobody → kritisingh1.ks
Blocks: 1520585
Status: NEW → ASSIGNED
Attachment #9050352 - Attachment is obsolete: true

Comment 9

2 months ago
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/44fee8919949
Set window handle for 'Switch To Window' also as 'handle'. r=whimboo

Comment 10

2 months ago
bugherder
Status: ASSIGNED → RESOLVED
Last Resolved: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.