Bug 47571

Summary: [Qt] AC rendering bugs
Product: WebKit Reporter: Girish Ramakrishnan <girish>
Component: WebKit QtAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, ademar, christian.webkit, commit-queue, eric, hausmann, noam, webkit.review.bot
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on:    
Bug Blocks: 35524    
Attachments:
Description Flags
Patch for the scroll issue
none
Patch: second rendering bug (clicking a plugin) none

Description Girish Ramakrishnan 2010-10-12 22:01:35 PDT
Bugs in Qt's AC implementation
1. When Scrolling the view, the items don't move correctly. This is reproducible using http://webkit.org/blog-files/pulse.html. Just make the view small enough and scroll vertically. The div gets clipped. 


2. For AC/plugins (see 35524), the plugin layer moves incorrectly when scrolling.

3. When clicking on the plugin, the plugin moves/increases in size.
Comment 1 Noam Rosenthal 2010-10-18 07:00:51 PDT
Created attachment 71030 [details]
Patch for the scroll issue
Comment 2 Noam Rosenthal 2010-10-18 07:09:03 PDT
Created attachment 71031 [details]
Patch: second rendering bug (clicking a plugin)
Comment 3 Andreas Kling 2010-10-18 07:51:28 PDT
Comment on attachment 71030 [details]
Patch for the scroll issue

LGTM.
Comment 4 Andreas Kling 2010-10-18 08:15:17 PDT
Comment on attachment 71031 [details]
Patch: second rendering bug (clicking a plugin)

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

LGTM.

> WebCore/platform/graphics/qt/GraphicsLayerQt.cpp:864
> +            return;

Nit: "break;" would fit better in this context.
Comment 5 Girish Ramakrishnan 2010-10-18 08:20:12 PDT
The scroll patch breaks youtube for me (scrolls fine but nothing renders).

Patches rebased over r69947
Comment 6 Girish Ramakrishnan 2010-10-18 08:28:54 PDT
Scrolling works after applying both patches. Can we commit them as a single patch (or split them correctly?)
Comment 7 Girish Ramakrishnan 2010-10-18 09:10:59 PDT
Err, sorry for the confusion. I had some patches before noam's patches. All good, commit in any order.
Comment 8 WebKit Commit Bot 2010-10-18 09:48:50 PDT
Comment on attachment 71030 [details]
Patch for the scroll issue

Clearing flags on attachment: 71030

Committed r69966: <http://trac.webkit.org/changeset/69966>
Comment 9 WebKit Commit Bot 2010-10-18 10:28:08 PDT
Comment on attachment 71031 [details]
Patch: second rendering bug (clicking a plugin)

Clearing flags on attachment: 71031

Committed r69971: <http://trac.webkit.org/changeset/69971>
Comment 10 WebKit Commit Bot 2010-10-18 10:28:15 PDT
All reviewed patches have been landed.  Closing bug.
Comment 11 WebKit Review Bot 2010-10-18 11:06:35 PDT
http://trac.webkit.org/changeset/69971 might have broken Qt Windows 32-bit Release
Comment 12 Ademar Reis 2010-10-21 13:34:37 PDT
Removing bug 39121 (qtwebkit-2.1 release critical meta bug) from the list of blockers because this change won't be integrated into qtwebkit-2.1 without the other changes related to bug 35524, which already blocks bug 39121