Bug 126459
| Summary: | Region is 432 bytes out of the box | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Tim Horton <thorton> |
| Component: | Platform | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | andersca, danakj, kling, koivisto, sam, simon.fraser |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Tim Horton
Region is enormous! It has Vectors with a huge inline capacity. We should do something to make it smaller (and maybe especially optimize for the case where it ends up just being one rect).
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Simon Fraser (smfr)
Loading webkit.org makes large regions:
Region 0x7fb766810f98: 208 segments, 84 spans
yahoo.com:
Region 0x7fb766811148: 442 segments, 76 spans
These big regions are from Page::addRelevantRepaintedObject().
Many regions are small, however.
Tim Horton
Is it possible to make it so that we can specify the inline size on the Region? Then we can make the relevant repainted object ones big and default to small?
Tim Horton
(like Vector)
Anders Carlsson
I think what we really want for tracking repaint rects is some sort of non-exact region that tries to stay simple...
Tim Horton
(In reply to comment #4)
> I think what we really want for tracking repaint rects is some sort of non-exact region that tries to stay simple...
That is probably true.