[webkit-changes] [231520] trunk/Source/WebCore

Tue, 08 May 2018 16:27:00 -0700

Title: [231520] trunk/Source/WebCore
Revision
231520
Author
[email protected]
Date
2018-05-08 16:26:39 -0700 (Tue, 08 May 2018)

Log Message

Switch some RELEASE_ASSERTS to plain debug ASSERTS in PlatformScreenMac.mm
https://bugs.webkit.org/show_bug.cgi?id=185451
<rdar://problem/39620348>

Reviewed by Zalan Bujtas.

Change a set of RELEASE_ASSERTS used to prevent accessing NSScreen related functions in the
PlatformScreenMac implementation to less expensive Debug ASSERTS.

No change in behavior.

* platform/mac/PlatformScreenMac.mm:
(WebCore::screenHasInvertedColors):
(WebCore::screenDepth):
(WebCore::screenDepthPerComponent):
(WebCore::screenRectForDisplay):
(WebCore::screenRect):
(WebCore::screenAvailableRect):
(WebCore::screenColorSpace):
(WebCore::screenSupportsExtendedColor):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (231519 => 231520)


--- trunk/Source/WebCore/ChangeLog	2018-05-08 23:22:06 UTC (rev 231519)
+++ trunk/Source/WebCore/ChangeLog	2018-05-08 23:26:39 UTC (rev 231520)
@@ -1,3 +1,26 @@
+2018-05-08  Brent Fulgham  <[email protected]>
+
+        Switch some RELEASE_ASSERTS to plain debug ASSERTS in PlatformScreenMac.mm
+        https://bugs.webkit.org/show_bug.cgi?id=185451
+        <rdar://problem/39620348>
+
+        Reviewed by Zalan Bujtas.
+
+        Change a set of RELEASE_ASSERTS used to prevent accessing NSScreen related functions in the
+        PlatformScreenMac implementation to less expensive Debug ASSERTS.
+
+        No change in behavior.
+
+        * platform/mac/PlatformScreenMac.mm:
+        (WebCore::screenHasInvertedColors):
+        (WebCore::screenDepth):
+        (WebCore::screenDepthPerComponent):
+        (WebCore::screenRectForDisplay):
+        (WebCore::screenRect):
+        (WebCore::screenAvailableRect):
+        (WebCore::screenColorSpace):
+        (WebCore::screenSupportsExtendedColor):
+
 2018-05-08  Daniel Bates  <[email protected]>
 
         Resign Strong Password appearance when text field value changes

Modified: trunk/Source/WebCore/platform/mac/PlatformScreenMac.mm (231519 => 231520)


--- trunk/Source/WebCore/platform/mac/PlatformScreenMac.mm	2018-05-08 23:22:06 UTC (rev 231519)
+++ trunk/Source/WebCore/platform/mac/PlatformScreenMac.mm	2018-05-08 23:26:39 UTC (rev 231520)
@@ -181,7 +181,7 @@
         return screenProperties(primaryScreenDisplayID()).screenHasInvertedColors;
 
     // This is a system-wide accessibility setting, same on all screens.
-    RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
+    ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
     return CGDisplayUsesInvertedPolarity();
 }
 
@@ -193,7 +193,7 @@
         return screenDepth;
     }
 
-    RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
+    ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
     return NSBitsPerPixelFromDepth(screen(widget).depth);
 }
 
@@ -205,7 +205,7 @@
         return depthPerComponent;
     }
 
-    RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
+    ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
     return NSBitsPerSampleFromDepth(screen(widget).depth);
 }
 
@@ -217,7 +217,7 @@
         return screenRect;
     }
 
-    RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
+    ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
     return screen(displayID).frame;
 }
 
@@ -226,7 +226,7 @@
     if (!screenProperties().isEmpty())
         return getScreenProperties(widget).screenRect;
 
-    RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
+    ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
     return toUserSpace([screen(widget) frame], window(widget));
 }
 
@@ -235,7 +235,7 @@
     if (!screenProperties().isEmpty())
         return getScreenProperties(widget).screenAvailableRect;
 
-    RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
+    ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
     return toUserSpace([screen(widget) visibleFrame], window(widget));
 }
 
@@ -260,7 +260,7 @@
     if (!screenProperties().isEmpty())
         return getScreenProperties(widget).colorSpace.get();
 
-    RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
+    ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
     return screen(widget).colorSpace.CGColorSpace;
 }
 
@@ -269,7 +269,7 @@
     if (!screenProperties().isEmpty())
         return getScreenProperties(widget).screenSupportsExtendedColor;
 
-    RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
+    ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
     return [screen(widget) canRepresentDisplayGamut:NSDisplayGamutP3];
 }
 

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to