Closed Bug 711867 Opened 8 years ago Closed 8 years ago

Track dirty region in gonk widget backend

Categories

(Core :: Widget, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla12

People

(Reporter: cjones, Assigned: mwu)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

See bug 708524.

There's an implementation of this for PuppetWidget.cpp that can be copied.  I'm not sure if it's worth sharing that impl yet.
Mike, can you take this?  This is pretty high priority; it's blocking measuring and fixing perf issues in b2g-gonk.
Assignee: nobody → mwu
Attached patch Track dirty region (obsolete) — Splinter Review
This also adds support for sync invalidation since we're already in here fixing this function.
Attachment #584850 - Flags: review?(jones.chris.g)
Comment on attachment 584850 [details] [diff] [review]
Track dirty region

Uh, screwed something up, lemme upload a new patch.
Attachment #584850 - Attachment is obsolete: true
Attachment #584850 - Flags: review?(jones.chris.g)
Attachment #584855 - Flags: review?(jones.chris.g)
Comment on attachment 584855 [details] [diff] [review]
Track dirty region, v2

Let's get this landed asap.  Will you be able to today?
Attachment #584855 - Flags: review?(jones.chris.g) → review+
We usually prevent regions from getting arbitrary complex in situations like this by using something like SimplifyOutward. Fodder for a future bug I guess.
You need to log in before you can comment on or make changes to this bug.