Title: [277833] trunk/Source/WebCore
Revision
277833
Author
wei...@apple.com
Date
2021-05-20 16:01:24 -0700 (Thu, 20 May 2021)

Log Message

Fix inverted ASSERT in sampleColor.

Rubber-stamped by Tim Horton.

* page/PageColorSampler.cpp:
(WebCore::sampleColor):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (277832 => 277833)


--- trunk/Source/WebCore/ChangeLog	2021-05-20 22:33:35 UTC (rev 277832)
+++ trunk/Source/WebCore/ChangeLog	2021-05-20 23:01:24 UTC (rev 277833)
@@ -1,3 +1,12 @@
+2021-05-20  Sam Weinig  <wei...@apple.com>
+
+        Fix inverted ASSERT in sampleColor.
+
+        Rubber-stamped by Tim Horton.
+
+        * page/PageColorSampler.cpp:
+        (WebCore::sampleColor):
+
 2021-05-20  Alexey Shvayka  <shvaikal...@gmail.com>
 
         [WebIDL] Remove [ImplicitThis] and [CustomProxyToJSObject] extended attributes

Modified: trunk/Source/WebCore/page/PageColorSampler.cpp (277832 => 277833)


--- trunk/Source/WebCore/page/PageColorSampler.cpp	2021-05-20 22:33:35 UTC (rev 277832)
+++ trunk/Source/WebCore/page/PageColorSampler.cpp	2021-05-20 23:01:24 UTC (rev 277833)
@@ -108,7 +108,7 @@
     if (!pixelBuffer)
         return WTF::nullopt;
 
-    ASSERT(pixelBuffer->data().length() <= 4);
+    ASSERT(pixelBuffer->data().length() >= 4);
 
     auto snapshotData = pixelBuffer->data().data();
     return convertColor<Lab<float>>(SRGBA<uint8_t> { snapshotData[2], snapshotData[1], snapshotData[0], snapshotData[3] });
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to