Patch for bug 237085 makes us call GetMaxChunkLength too much

RESOLVED WONTFIX

Status

Core Graveyard
GFX
RESOLVED WONTFIX
12 years ago
4 years ago

People

(Reporter: bz, Unassigned)

Tracking

({perf})

Dependency tree / graph
Bug Flags:
blocking1.9 -

Firefox Tracking Flags

(Not tracked)

Details

The patch for bug 237085 added calls to GetMaxChunkLength to the various GFX apis.  Unfortunately, some API functions call other API functions.  :(  Typical example:

nsRenderingContextImpl::GetTextDimensions calls GetMaxChunkLength.

It also calls nsThebesRenderingContext::GetTextDimensionsInternal which calls nsThebesRenderingContext::GetWidth which calls nsRenderingContextImpl::GetWidth which calls GetMaxChunkLength.

Perhaps GetTextDimensionsInternal should be calling GetWidthInternal directly?  Or something like that?
Flags: blocking1.9?
Summary: Patch for bug 237085 makes us call GetMaxChunkLength to much → Patch for bug 237085 makes us call GetMaxChunkLength too much
But, this code won't live long on trunk, so this is only a branch issue.
I don't really care about branch.  This function is pretty fast except with cairo.  I assume on trunk the textframe rewrite is making this go away?
Depends on: 351242

Updated

12 years ago
No longer depends on: 351242
code going away, so not blocking
Flags: blocking1.9? → blocking1.9-
Product: Core → Core Graveyard
This bug has been buried in the graveyard and has not been updated in over 5 years. It is probably safe to assume that it will never be fixed, so resolving as WONTFIX.

[Mass-change filter: graveyard-wontfix-2014-09-24]
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.