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

Reply via email to