Bug 302294 (longlines)

[meta] Bugs in rendering of long text strings (e.g. thousands of characters)

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
14 years ago
8 years ago

People

(Reporter: jruderman, Unassigned)

Tracking

({meta})

Trunk
PowerPC
macOS
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Reporter

Description

14 years ago
There are many bugs that occur when Firefox tries to render a large amount of
text (e.g. thousands of characters). Common themes are severe slowdowns, system
freezes, overlapping text, and invisible text.  Some of these issues might
impact security -- overlapping or invisible text could lead to spoofing or at
least users having less information than they should, and system freezes are
considered a security issue even though they're really the fault of the
operating system.
There's a workaround for many of these in the Xft font code, but not other font
backends.  Probably we should just do something in nsTextFrame -- perhaps even
just creating a maximum text frame size (say, 512 characters, which is I think
what Xft uses) and using next-in-flow linkage for text frames within the same
line.  While it might be nice to want the Gfx API to be pure, it's probably a
lot easier to restrict it and require layout to put things in chunks.  We should
be careful not to separate grapheme clusters between chunks -- i.e., be careful
of both surrogates (representing characters outside of Unicode Plane 0) and
combining characters (which are a bit harder).
Reporter

Updated

14 years ago
Whiteboard: [sg:investigate]
Reporter

Updated

14 years ago
Depends on: 292667
Reporter

Updated

14 years ago
Depends on: 303546
Reporter

Updated

14 years ago
Depends on: 303819
Reporter

Updated

14 years ago
Depends on: 303925
Reporter

Updated

14 years ago
Depends on: 305144
Reporter

Updated

14 years ago
Depends on: 305264
Reporter

Updated

14 years ago
Depends on: 306529

Updated

14 years ago
No longer depends on: 303925
Reporter

Updated

14 years ago
Depends on: 265551
Reporter

Updated

14 years ago
Depends on: 316874

Updated

14 years ago
No longer depends on: 305144

Updated

14 years ago
Blocks: 321142

Updated

14 years ago
No longer blocks: 321142

Updated

14 years ago
Depends on: 322920
Reporter

Updated

14 years ago
Depends on: 323385
Reporter

Updated

14 years ago
Depends on: 326191
Reporter

Updated

14 years ago
Depends on: 326188
Reporter

Updated

14 years ago
Depends on: 326176
*** Bug 326229 has been marked as a duplicate of this bug. ***
Reporter

Updated

13 years ago
Depends on: 331086
Reporter

Updated

13 years ago
Depends on: 334814
Reporter

Updated

13 years ago
Depends on: 338251
Reporter

Updated

13 years ago
Depends on: 338510
Reporter

Updated

13 years ago
Depends on: 295769
Reporter

Updated

13 years ago
No longer depends on: 338510
I'm working on something along the lines of comment #1. But I don't want to change how textframes break, that's rather risky. Instead I'm defining some wrapper functions around the nsIRenderingContext functions that break the text up into chunks of limited size.
Reporter

Comment 4

13 years ago
Bug 338251 has a patch that fixes most of these issues.
Reporter

Comment 5

13 years ago
The patch in bug 338251 has been checked in.  Bug 237085 has a patch that fixes some more issues.
Reporter

Updated

13 years ago
Depends on: 342922
Alias: longlines
Reporter

Updated

13 years ago
Depends on: 348202

Updated

13 years ago
Depends on: 346234
Reporter

Updated

13 years ago
Depends on: 332881
Reporter

Updated

13 years ago
Depends on: 361612
Reporter

Updated

13 years ago
Depends on: 363061
Someone want to retest these bugs on trunk? Most of them should be fixed now.
Reporter

Updated

11 years ago
Depends on: 452559
Product: Core → Core Graveyard
Depends on: 269050, 510688
Depends on: 350993
Depends on: 458882
Reporter

Updated

10 years ago
Depends on: 266140
Reporter

Updated

9 years ago
Assignee: general → nobody
Component: GFX → Graphics
Product: Core Graveyard → Core
QA Contact: ian → thebes
Whiteboard: [sg:investigate]
Reporter

Updated

9 years ago
Depends on: 594637
I've been going through the tracked bug here and I can't reproduce any of these issues. Closing this tracking bug.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.