Bug 46716

Summary: Full-page Adobe Reader does not paint until window is resized
Product: WebKit Reporter: Adam Roben (:aroben) <aroben>
Component: Plug-insAssignee: Nobody <webkit-unassigned>
Severity: Normal CC: andersca, jhoneycutt, webkit.review.bot
Priority: P2 Keywords: InRadar, PlatformOnly
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
URL: http://www.longwood.edu/helpdesk/blackboard/Bbfiletypes/pdffiles.pdf
Description Flags
Don't call NPP_SetWindow until the plugin's HWND has been sized/positioned andersca: review+

Description Adam Roben (:aroben) 2010-09-28 04:30:53 PDT
To reproduce:

1. Go to http://www.longwood.edu/helpdesk/blackboard/Bbfiletypes/pdffiles.pdf
2. Resize the window

Adobe Reader doesn't paint until step 2.

Comment 1 Adam Roben (:aroben) 2010-09-28 04:38:02 PDT
Created attachment 69038 [details]
Don't call NPP_SetWindow until the plugin's HWND has been sized/positioned
Comment 2 WebKit Review Bot 2010-09-28 04:43:08 PDT
Attachment 69038 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
WebKitTools/DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:63:  PluginTest::NPP_SetWindow is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
WebKitTools/DumpRenderTree/TestNetscapePlugIn/Tests/win/WindowGeometryInitializedBeforeSetWindow.cpp:26:  Found other header before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted.  [build/include_order] [4]
WebKitTools/DumpRenderTree/TestNetscapePlugIn/Tests/win/WindowGeometryInitializedBeforeSetWindow.cpp:28:  Alphabetical sorting problem.  [build/include_order] [4]
Total errors found: 3 in 11 files

If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Adam Roben (:aroben) 2010-09-28 08:45:13 PDT
Committed r68520: <http://trac.webkit.org/changeset/68520>