The default bug view has changed. See this FAQ.

Optimize textframe handling of a single large textrun during a horizontal resize

RESOLVED FIXED in mozilla9

Status

()

Core
Layout
P1
normal
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: bz, Assigned: bz)

Tracking

(Blocks: 2 bugs)

unspecified
mozilla9
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

This is spun off from bug 561578 to handle the textframe parts of that bug.  The blockframe parts are more painful, sadly.
Blocks: 680461
Created attachment 555808 [details] [diff] [review]
Optimize textframe to handle horizontal resizes of large runs of text better.   change does two things:
Attachment #555808 - Flags: review?(roc)
Whiteboard: [need review]
Comment on attachment 555808 [details] [diff] [review]
Optimize textframe to handle horizontal resizes of large runs of text better.   change does two things:

Review of attachment 555808 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/generic/nsTextFrameThebes.cpp
@@ +6680,5 @@
>    return PR_FALSE;
>  }
>  
> +static void
> +RemoveInFlows(nsIFrame* f, nsIFrame* firstToNotRemove)

aFrame, aFirstToNotRemove
Attachment #555808 - Flags: review?(roc) → review+
Made those changes and pushed http://hg.mozilla.org/integration/mozilla-inbound/rev/171e1d0b8cb3

Not sure how to write a test for this....  It's way too likely to regress.  :(
Whiteboard: [need review]
Target Milestone: --- → mozilla9
http://hg.mozilla.org/mozilla-central/rev/171e1d0b8cb3
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.