Changeset 248028 in webkit

Timestamp:

Jul 30, 2019, 10:01:42 PM (7 years ago)

Author:

timothy_horton@apple.com

Message:

ASSERTion failure under takeSnapshot after r247846

• page/TextIndicator.cpp:

(WebCore::takeSnapshots):
We now sometimes inflate the scale factor; allow this.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/TextIndicator.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2019-07-30  Tim Horton  <timothy_horton@apple.com>
+
+        ASSERTion failure under takeSnapshot after r247846
+
+        * page/TextIndicator.cpp:
+        (WebCore::takeSnapshots):
+        We now sometimes inflate the scale factor; allow this.
+
 2019-07-30  Saam Barati  <sbarati@apple.com>
 

trunk/Source/WebCore/page/TextIndicator.cpp

@@ -170 +170 @@
         float snapshotScaleFactor;
         data.contentImageWithHighlight = takeSnapshot(frame, snapshotRect, SnapshotOptionsNone, snapshotScaleFactor, clipRectsInDocumentCoordinates);
-        ASSERT(!data.contentImageWithHighlight || data.contentImageScaleFactor == snapshotScaleFactor);
+        ASSERT(!data.contentImageWithHighlight || data.contentImageScaleFactor >= snapshotScaleFactor);
     }