Bug 36479 - Plug-in clip rect does not update when overflow clip changes
Summary: Plug-in clip rect does not update when overflow clip changes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: 528+ (Nightly build)
Hardware: Macintosh OS X 10.6
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2010-03-22 22:53 PDT by mitz
Modified: 2010-03-23 17:28 PDT (History)
3 users (show)

See Also:


Attachments
Notify views of visible-rect-only changes (9.92 KB, patch)
2010-03-22 22:55 PDT, mitz
sullivan: review+
Details | Formatted Diff | Diff
Notify views of visible-rect-only changes (9.92 KB, patch)
2010-03-22 23:05 PDT, mitz
sullivan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2010-03-22 22:53:24 PDT
<rdar://problem/7197736>

When changes to overflow clip affect a plug-in’s visible rect, but its frame rect doesn’t change, if the plug-in is not in a compositing layer, then it does not get updated to reflect the change to clipping. This is demonstrated by a manual test which is part of a forthcoming patch.
Comment 1 mitz 2010-03-22 22:55:33 PDT
Created attachment 51395 [details]
Notify views of visible-rect-only changes
Comment 2 WebKit Review Bot 2010-03-22 23:00:56 PDT
Attachment 51395 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKit/mac/Plugins/WebBaseNetscapePluginView.h:52:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 1 in 11 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Eric Seidel (no email) 2010-03-22 23:01:47 PDT
Attachment 51395 [details] did not build on mac:
Build output: http://webkit-commit-queue.appspot.com/results/981123
Comment 4 mitz 2010-03-22 23:05:42 PDT
Created attachment 51396 [details]
Notify views of visible-rect-only changes
Comment 5 WebKit Review Bot 2010-03-22 23:06:47 PDT
Attachment 51396 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKit/mac/Plugins/WebBaseNetscapePluginView.h:52:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 1 in 11 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 Simon Fraser (smfr) 2010-03-23 09:06:29 PDT
Comment on attachment 51396 [details]
Notify views of visible-rect-only changes


> Index: WebCore/manual-tests/plugin-visible-rect-change.html
> ===================================================================
> --- WebCore/manual-tests/plugin-visible-rect-change.html	(revision 0)
> +++ WebCore/manual-tests/plugin-visible-rect-change.html	(revision 0)
> @@ -0,0 +1,25 @@
> +<p>
> +    This test requires that accelerated compositing be dsiabled

Typo: "dsiabled". I'll review more later.
Comment 7 John Sullivan 2010-03-23 09:41:15 PDT
Oops, I r+'ed the obsolete version.
Comment 8 John Sullivan 2010-03-23 09:44:55 PDT
Comment on attachment 51396 [details]
Notify views of visible-rect-only changes

> +    This test requires that accelerated compositing be dsiabled. Clicking the Test button should completely reveal the black-bordered box.
> +    The orange video should completely fill the box, with the controller visible and touching the bottom of the box. There should be
> +    not ba any red visible.

r+ if you fix "dsiabled" and "ba any red".
Comment 9 mitz 2010-03-23 17:28:40 PDT
Fixed in <http://trac.webkit.org/projects/webkit/changeset/56429> except I forgot to fix the typos in the test :-(