Bug 87062 - [TextureMapper] [Cairo] Implement repaint counters
Summary: [TextureMapper] [Cairo] Implement repaint counters
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Martin Robinson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-21 18:00 PDT by Martin Robinson
Modified: 2012-05-22 15:09 PDT (History)
3 users (show)

See Also:


Attachments
Patch (4.52 KB, patch)
2012-05-21 18:29 PDT, Martin Robinson
gustavo: review+
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Robinson 2012-05-21 18:00:28 PDT
Repaint counters can be useful for debugging content that is composited with AC and AC implementations.
Comment 1 Martin Robinson 2012-05-21 18:29:47 PDT
Created attachment 143160 [details]
Patch
Comment 2 Martin Robinson 2012-05-21 18:32:14 PDT
This code in Cairo only, but if the Qt port is interested in this feature, I'm willing to write it using QPainter. The reason I did not use generic GraphicsContext APIs it that it would require some interaction with the FontCache and using the toy platform interface seems simpler all around.
Comment 3 Noam Rosenthal 2012-05-22 00:03:00 PDT
Comment on attachment 143160 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=143160&action=review

I'm ok with the texture-mapper changes, though I'm not qualified to review the cairo-specific code :)
We'd love a QPainter implementation of this, though we'd have to do this in the LayerBackingStore code which is the proper place in WebKit2.

> Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp:573
> +#if USE(CAIRO)

This would be more readable inside the function.
Comment 4 Gustavo Noronha (kov) 2012-05-22 10:44:52 PDT
Comment on attachment 143160 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=143160&action=review

I'm just not sure about the border color, feels like you may want to have a border distinct from the background, but I guess it'll look readable like this anyway.

>> Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp:573
>> +#if USE(CAIRO)
> 
> This would be more readable inside the function.

Hmm, I think it would be less readable with different implementations taking the whole function body. As it is now it's pretty obvious that each implementation has a completely different implementation.
Comment 5 WebKit Review Bot 2012-05-22 12:22:27 PDT
Comment on attachment 143160 [details]
Patch

Rejecting attachment 143160 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
webkit-commit-queue/Source/WebKit/chromium/webkit --revision 138079 --non-interactive --force --accept theirs-conflict --ignore-externals' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
48>At revision 138079.

________ running '/usr/bin/python tools/clang/scripts/update.py --mac-only' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'

________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
Updating webkit projects from gyp files...

Full output: http://queues.webkit.org/results/12763103
Comment 6 Martin Robinson 2012-05-22 15:09:08 PDT
Committed r118061: <http://trac.webkit.org/changeset/118061>