If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Form helper can't navigate in nested frames

RESOLVED FIXED

Status

Fennec Graveyard
General
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: Martijn Wargers (dead), Assigned: vingtetun)

Tracking

({testcase})

Trunk
x86
Windows 7
testcase

Details

(Whiteboard: formfill)

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

8 years ago
Created attachment 443696 [details]
testcase

See testcase, when clicking on the first text input, the form helper comes up.
But notice that you can't click on the "Next" button, which should be possible in that case.
Whiteboard: formfill
Vivien, can you take a look?

Martijn, what real world use case?
(Reporter)

Comment 2

8 years ago
I didn't find this on a site, just during experimentation.
Created attachment 445342 [details] [diff] [review]
Patch

Tests is coming! :)
Attachment #445342 - Flags: review?(mark.finkle)
Created attachment 445369 [details] [diff] [review]
Patch + tests

Patch + tests
Assignee: nobody → 21
Attachment #445342 - Attachment is obsolete: true
Attachment #445369 - Flags: review?(mark.finkle)
Attachment #445342 - Flags: review?(mark.finkle)
Comment on attachment 445369 [details] [diff] [review]
Patch + tests

>diff -r 9772d862d2da chrome/content/browser-ui.js

>+  _getAllDocuments: function formHelper_getAllDocuments(aDocument, aResult) {
>+    /** Recursively find all documents, including root document. */

Remove this comment. The code tells the story pretty well.
Attachment #445369 - Flags: review?(mark.finkle) → review+
http://hg.mozilla.org/mobile-browser/rev/431478c437f1
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.