Bug 72425

Summary: Clip rects should be in root view coordinates
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: NEW ---    
Severity: Normal CC: ahmad.saleem792, ap, bfulgham, dglazkov, simon.fraser, webkit.review.bot, zalan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch sam: review+

Description Anders Carlsson 2011-11-15 14:36:48 PST
Clip rects should be in root view coordinates
Comment 1 Anders Carlsson 2011-11-15 14:41:52 PST
Created attachment 115251 [details]
Patch
Comment 2 WebKit Review Bot 2011-11-15 15:45:36 PST
Comment on attachment 115251 [details]
Patch

Attachment 115251 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10483495
Comment 3 Anders Carlsson 2011-11-15 17:25:39 PST
Created attachment 115285 [details]
Patch
Comment 4 Ahmad Saleem 2022-10-05 16:16:50 PDT
From Webkit GitHub linkage to attach patch modification:

EmbeddedWidget.cpp is no more.

FrameView.cpp - It is still to Legacy - https://github.com/WebKit/WebKit/blob/71af797599da12fa56a502b6f23501f31e289dc2/Source/WebCore/page/FrameView.cpp#L4063

then there are some Chromium build changes - which are not applicable anymore.

In PluginView.cpp - we still have this (more of renaming it seems in whole patch):

https://github.com/WebKit/WebKit/blob/23e2639c3e05df5f20cb2a3b5411ee5f8b846206/Source/WebKit/WebProcess/Plugins/PluginView.cpp#L659

widgetPositionsUpdated is removed from RenderWidget.cpp:

https://github.com/WebKit/WebKit/commit/a0b9903d75fd1b579789122ffaa728aab69bf016

____

In short summary, do we need this change? Thanks!