357544 - [reflow branch] Frame(wbr)(0) frame didn't implement GetPrefWidth/GetMinWidth

Status: VERIFIED FIXED

(Core :: Layout: Block and Inline, defect, P2)

Description

[Mats Palmgren (inactive)]

STEPS TO REPRODUCE
1. load the attached testcase in a debug build

ACTUAL RESULT
WARNING: Frame(wbr)(0) frame didn't implement GetPrefWidth: file nsFrame.cpp, line 2846
WARNING: Frame(wbr)(0) frame didn't implement GetMinWidth: file nsFrame.cpp, line 2832

Comment 1

[Mats Palmgren (inactive)]

Attachment: Testcase #1

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

wbr elements just get an nsFrame created for them.  Given that, I should probably remove this warning -- it was there mainly to help me during development.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

But I might want to leave it for a bit longer...

Comment 4

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

I get the same warning for toolbarspring

Comment 6

[Boris Zbarsky [:bzbarsky]]

We could also create a subclass of nsLeafFrame that returns 0 for the sizes and use that for wbr...

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: remove the warnings

Comment 8

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Checked in to trunk.

Comment 9

[Samuel Sidler (old account; do not CC)]

v.fixed through proxy with ctalbert's debug build