tree 3b6d6c962145e05268ec72d428e1f81fb037475a
parent 9648e842316b49b4bc383b5a030ecb687da09070
author Lei Zhang <thestig@chromium.org> 1582769096 +0000
committer Chromium commit bot <commit-bot@chromium.org> 1582769096 +0000

Handle case of nothing to restore in SkiaState::ClipRestore().

Currently, ClipRestore() looks through the |m_commands| stack for a
save command to restore. If there is no save command in the stack,
ClipRestore() will hit an assertion failure, or go out of bounds if
assertions are disabled.

Restore operations with no corresponding save happens occasionally. AGG
handles this case properly, so do something similar in Skia. While
looking through |m_commands| for a save command to restore, if one is
not found, stop looking and just return. Effectively turning
ClipRestore() into a no-op.

Bug: chromium:986108
Change-Id: I53e9ad831f0dbfe76ad7978361f5c5e928661f30
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/66874
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Mike Reed <reed@google.com>