Add tests to ensure changing number of rows or columns in a frameset works as expected

RESOLVED FIXED in Firefox 56

Status

()

Core
DOM
P3
normal
RESOLVED FIXED
6 months ago
6 months ago

People

(Reporter: bytesized, Assigned: bytesized)

Tracking

unspecified
mozilla56
Points:
---

Firefox Tracking Flags

(firefox56 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

6 months ago
I noticed that some incorrect code that I wrote where (effectively) |HTMLFrameSetElement::mCurrentRowColHint| was always set to |NS_STYLE_HINT_REFLOW| broke no tests.

Manual tests for this already exist at these locations:
 - https://bug48422.bmoattachments.org/attachment.cgi?id=90746
 - https://bug48422.bmoattachments.org/attachment.cgi?id=90747

When these tests are run with the buggy code I wrote, the frames seem to be added fine, but when removed, nothing is displayed on the page.
Priority: -- → P3
Depends on: 1365092
(Assignee)

Updated

6 months ago
Assignee: nobody → ksteuber
Comment hidden (mozreview-request)
(Assignee)

Updated

6 months ago
Attachment #8877788 - Flags: review?(bzbarsky)
Comment on attachment 8877788 [details]
Bug 1371073 - Add reftests to ensure rows and cols are properly added and removed from framesets

https://reviewboard.mozilla.org/r/149214/#review154304

r=me.

Did you happen to check whether some of these fail with the buggy bits from bug 1365092?
Attachment #8877788 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 3

6 months ago
I did check and the "remove" tests both fail in that situation.

Comment 4

6 months ago
Pushed by ksteuber@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9afdcc193530
Add reftests to ensure rows and cols are properly added and removed from framesets r=bz

Comment 5

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9afdcc193530
Status: NEW → RESOLVED
Last Resolved: 6 months ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.