Bug 54562 - Web Inspector: remove platform specifics from node highlight routines.
Summary: Web Inspector: remove platform specifics from node highlight routines.
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-16 09:02 PST by Pavel Feldman
Modified: 2011-08-08 22:47 PDT (History)
17 users (show)

See Also:


Attachments
Patch (12.93 KB, patch)
2011-02-16 09:57 PST, Pavel Feldman
no flags Details | Formatted Diff | Diff
Patch (12.97 KB, patch)
2011-02-16 10:03 PST, Pavel Feldman
pfeldman: review-
Details | Formatted Diff | Diff
[PATCH] For bots (75.66 KB, patch)
2011-02-16 10:45 PST, Pavel Feldman
no flags Details | Formatted Diff | Diff
Patch (40.90 KB, patch)
2011-05-31 13:55 PDT, Vsevolod Vlasov
simon.fraser: review-
gustavo.noronha: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Feldman 2011-02-16 09:02:18 PST
We should be able to handle everything in webcore. First patch makes everything work in WebCore, second will remove WebKit code.
Comment 1 Pavel Feldman 2011-02-16 09:57:00 PST
Created attachment 82655 [details]
Patch
Comment 2 Pavel Feldman 2011-02-16 10:03:04 PST
Created attachment 82658 [details]
Patch
Comment 3 Pavel Feldman 2011-02-16 10:45:01 PST
Created attachment 82662 [details]
[PATCH] For bots
Comment 4 Early Warning System Bot 2011-02-16 11:05:32 PST
Attachment 82662 [details] did not build on qt:
Build output: http://queues.webkit.org/results/7920378
Comment 5 Build Bot 2011-02-16 11:13:13 PST
Attachment 82662 [details] did not build on win:
Build output: http://queues.webkit.org/results/7918376
Comment 6 Pavel Feldman 2011-02-16 11:24:30 PST
Comment on attachment 82658 [details]
Patch

Ok, so I now have a better understanding of why it was done in WebKit: compositing. James R. hinted that we could try introducing a separate layer on top of the world with the highlight. I'll look into it tomorrow. It should be all WebCore (rendering/ and platform/graphics/*).
Comment 7 Vsevolod Vlasov 2011-05-31 13:55:09 PDT
Created attachment 95477 [details]
Patch
Comment 8 Vsevolod Vlasov 2011-05-31 13:57:58 PDT
This is a first attempt.
I'd like to get some high level review on the approach.

Also for some reason this doesn't work on mac non-WK2, whenever I try to highlight something in composited mode - highlight works fine, but all transformed layers disappear.

I did not made anything about WK2 yet.

see also: https://bugs.webkit.org/show_bug.cgi?id=54618
Comment 9 Collabora GTK+ EWS bot 2011-05-31 17:52:36 PDT
Comment on attachment 95477 [details]
Patch

Attachment 95477 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/8751808
Comment 10 Simon Fraser (smfr) 2011-06-06 07:56:53 PDT
Comment on attachment 95477 [details]
Patch

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

> Source/WebCore/rendering/RenderLayerCompositor.cpp:1474
> +    } else if (graphicsLayer == m_overlayLayer)
> +        m_renderView->frameView()->frame()->page()->pageOverlay()->paintComposited(context);

This is just too convoluted. RLC shouldn't have to know anything about page()->pageOverlay().

I still think that the overlay layer should be hooked up outside of WebCore, just like WK2 does it.
Comment 11 Pavel Feldman 2011-08-08 22:47:26 PDT
As per comment #10