stylo: RecoverLetterFrames doesn't play nicely with ::first-line

RESOLVED FIXED in Firefox 57

Status

()

P3
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Tracking

(Blocks: 1 bug)

53 Branch
mozilla57
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 fixed)

Details

Attachments

(1 attachment)

Something I just realized about the patches from bug 1324619: they don't handle the case when a ::first-letter is created directly inside a ::first-line instead of being wrapped in it.  This happens in RecoverLetterFrames.

Simple testcase:

  <style>
    div { border: 5px solid transparent; }
    div::first-line { }
    div::first-letter { border: inherit; border-color: yellow; }
  </style>
  <div onclick="this.appendChild(document.createTextNode('Does the first letter still have a border?'))">
    Click me.
  </div>
(Assignee)

Updated

a year ago
Depends on: 1324619
(Assignee)

Updated

a year ago
Flags: needinfo?(bzbarsky)
(Assignee)

Updated

a year ago
Priority: -- → P3
(Assignee)

Updated

a year ago
Blocks: 1388877
Comment hidden (mozreview-request)
(Assignee)

Updated

a year ago
Flags: needinfo?(bzbarsky)
Attachment #8896036 - Flags: review?(emilio+bugs) → review?(cam)

Comment 2

a year ago
mozreview-review
Comment on attachment 8896036 [details]
Bug 1385656.  Fix the interaction of RecoverLetterFrames and ::first-line.

https://reviewboard.mozilla.org/r/167308/#review172614

(Stealing per bz's request.)
Attachment #8896036 - Flags: review?(cam) → review+

Comment 3

a year ago
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/63ebcdc4d3ab
Fix the interaction of RecoverLetterFrames and ::first-line.  r=heycam

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/63ebcdc4d3ab
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Depends on: 1398581
You need to log in before you can comment on or make changes to this bug.