Title: [203224] trunk/Source/WebCore
Revision
203224
Author
commit-qu...@webkit.org
Date
2016-07-14 08:48:12 -0700 (Thu, 14 Jul 2016)

Log Message

[GLib] Use a GSource instead of a thread to poll memory pressure eventFD in linux implementation
https://bugs.webkit.org/show_bug.cgi?id=159346

Patch by Carlos Garcia Campos <cgar...@igalia.com> on 2016-07-14
Reviewed by Antonio Gomes.

This is a follow up of r203216 to fix wrong use of Optional values.

* platform/linux/MemoryPressureHandlerLinux.cpp:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (203223 => 203224)


--- trunk/Source/WebCore/ChangeLog	2016-07-14 12:33:35 UTC (rev 203223)
+++ trunk/Source/WebCore/ChangeLog	2016-07-14 15:48:12 UTC (rev 203224)
@@ -1,3 +1,14 @@
+2016-07-14  Carlos Garcia Campos  <cgar...@igalia.com>
+
+        [GLib] Use a GSource instead of a thread to poll memory pressure eventFD in linux implementation
+        https://bugs.webkit.org/show_bug.cgi?id=159346
+
+        Reviewed by Antonio Gomes.
+
+        This is a follow up of r203216 to fix wrong use of Optional values.
+
+        * platform/linux/MemoryPressureHandlerLinux.cpp:
+
 2016-07-14  Youenn Fablet  <you...@apple.com>
 
         DOM value iterable interfaces should use Array prototype methods

Modified: trunk/Source/WebCore/platform/linux/MemoryPressureHandlerLinux.cpp (203223 => 203224)


--- trunk/Source/WebCore/platform/linux/MemoryPressureHandlerLinux.cpp	2016-07-14 12:33:35 UTC (rev 203223)
+++ trunk/Source/WebCore/platform/linux/MemoryPressureHandlerLinux.cpp	2016-07-14 15:48:12 UTC (rev 203224)
@@ -183,11 +183,11 @@
     if (log)
         LOG(MemoryPressure, "%s, error : %m", log);
 
-    if (!m_eventFD) {
+    if (m_eventFD) {
         close(m_eventFD.value());
         m_eventFD = Nullopt;
     }
-    if (!m_pressureLevelFD) {
+    if (m_pressureLevelFD) {
         close(m_pressureLevelFD.value());
         m_pressureLevelFD = Nullopt;
     }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to