Title: [214412] trunk/Source/WebKit2
- Revision
- 214412
- Author
- [email protected]
- Date
- 2017-03-27 06:38:34 -0700 (Mon, 27 Mar 2017)
Log Message
[GTK] NetscapePluginX11::m_windowID is unused/unneeded
https://bugs.webkit.org/show_bug.cgi?id=170115
Patch by Adrian Perez de Castro <[email protected]> on 2017-03-27
Reviewed by Michael Catanzaro.
* WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:
(WebKit::NetscapePluginX11::NetscapePluginX11):
Remove the unneeded NetscapePluginX11::m_windowID member.
* WebProcess/Plugins/Netscape/x11/NetscapePluginX11.h:
Remove the unneeded NetscapePluginX11::m_windowID member.
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (214411 => 214412)
--- trunk/Source/WebKit2/ChangeLog 2017-03-27 09:37:47 UTC (rev 214411)
+++ trunk/Source/WebKit2/ChangeLog 2017-03-27 13:38:34 UTC (rev 214412)
@@ -1,3 +1,16 @@
+2017-03-27 Adrian Perez de Castro <[email protected]>
+
+ [GTK] NetscapePluginX11::m_windowID is unused/unneeded
+ https://bugs.webkit.org/show_bug.cgi?id=170115
+
+ Reviewed by Michael Catanzaro.
+
+ * WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:
+ (WebKit::NetscapePluginX11::NetscapePluginX11):
+ Remove the unneeded NetscapePluginX11::m_windowID member.
+ * WebProcess/Plugins/Netscape/x11/NetscapePluginX11.h:
+ Remove the unneeded NetscapePluginX11::m_windowID member.
+
2017-03-25 Andy Estes <[email protected]>
[iOS] Use snapshotting instead of printing to draw single-page PDFs
Modified: trunk/Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp (214411 => 214412)
--- trunk/Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp 2017-03-27 09:37:47 UTC (rev 214411)
+++ trunk/Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp 2017-03-27 13:38:34 UTC (rev 214412)
@@ -175,7 +175,6 @@
NetscapePluginX11::NetscapePluginX11(NetscapePlugin& plugin, Display* display, uint64_t windowID)
: m_plugin(plugin)
, m_pluginDisplay(display)
- , m_windowID(windowID)
{
// It seems flash needs the socket to be in the same process,
// I guess it uses gdk_window_lookup(), so we create a new socket here
Modified: trunk/Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.h (214411 => 214412)
--- trunk/Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.h 2017-03-27 09:37:47 UTC (rev 214411)
+++ trunk/Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.h 2017-03-27 13:38:34 UTC (rev 214412)
@@ -64,7 +64,6 @@
WebCore::XUniquePixmap m_drawable;
NPSetWindowCallbackStruct m_setWindowCallbackStruct;
#if PLATFORM(GTK)
- uint64_t m_windowID { 0 };
unsigned long m_npWindowID { 0 };
GtkWidget* m_platformPluginWidget { nullptr };
#endif
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes