Bug 121600

Summary: [GTK][WK2] A plugin dirtyRect within iframe remains after moving to a different page.
Product: WebKit Reporter: ChangSeok Oh <changseok>
Component: WebKitGTKAssignee: ChangSeok Oh <changseok>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, gustavo, mrobinson
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Linux   
Attachments:
Description Flags
TestCase
none
Patch
gustavo: review+
Patch none

ChangSeok Oh
Reported 2013-09-19 03:25:01 PDT
Created attachment 212046 [details] TestCase Find the attached file which shows the bug.
Attachments
TestCase (22.82 KB, application/gzip)
2013-09-19 03:25 PDT, ChangSeok Oh 		no flags
Details
Patch (1.59 KB, patch)
2013-09-20 02:54 PDT, ChangSeok Oh 		gustavo: review+
DetailsFormatted DiffDiff
Patch (1.65 KB, patch)
2013-09-23 12:13 PDT, ChangSeok Oh 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
ChangSeok Oh
Comment 1 2013-09-20 02:54:43 PDT
Created attachment 212141 [details] Patch
Gustavo Noronha (kov)
Comment 2 2013-09-23 05:55:44 PDT
Comment on attachment 212141 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=212141&action=review > Source/WebKit2/ChangeLog:8 > + Nowhere to destory NetscapePlugin::m_platformPluginWidget even after destroying destory -> destroy > Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:304 > +#if PLATFORM(GTK) > + if (m_platformPluginWidget) > + gtk_widget_destroy(m_platformPluginWidget); > +#endif The drawable is set to 0, do you know why is that? I think we should do this here too, for consistency in any case.
ChangSeok Oh
Comment 3 2013-09-23 12:13:23 PDT
Created attachment 212377 [details] Patch
WebKit Commit Bot
Comment 4 2013-09-23 13:08:33 PDT
Comment on attachment 212377 [details] Patch Clearing flags on attachment: 212377 Committed r156287: <http://trac.webkit.org/changeset/156287>
Note You need to log in before you can comment on or make changes to this bug.