Bug 201530

Summary:

Marking up a note on iOS results in a PDF with no contents

Product:

WebKit

Reporter:

Tim Horton <thorton>

Component:

New Bugs

Assignee:

Tim Horton <thorton>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, aestes, cdumez, commit-queue, simon.fraser, webkit-bug-importer

Priority:

P2

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	none

Tim Horton

Reported 2019-09-05 18:52:27 PDT

Marking up a note on iOS results in a PDF with no contents

Attachments
Patch (15.68 KB, patch) 2019-09-05 18:52 PDT, Tim Horton	no flags	Details Formatted Diff Diff
Patch (15.71 KB, patch) 2019-09-06 00:08 PDT, Tim Horton	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Tim Horton

Comment 1 2019-09-05 18:52:43 PDT

Created attachment 378150 [details] Patch

Tim Horton

Comment 2 2019-09-05 18:52:45 PDT

<rdar://problem/53686019>

Tim Horton

Comment 3 2019-09-06 00:08:07 PDT

Created attachment 378163 [details] Patch

WebKit Commit Bot

Comment 4 2019-09-06 11:46:21 PDT

Comment on attachment 378163 [details] Patch Clearing flags on attachment: 378163 Committed r249584: <https://trac.webkit.org/changeset/249584>

WebKit Commit Bot

Comment 5 2019-09-06 11:46:22 PDT

All reviewed patches have been landed. Closing bug.

Alex Christensen

Comment 6 2019-09-07 15:17:02 PDT

The API test is asserting: https://build.webkit.org/builders/Apple%20iOS%2012%20Simulator%20Debug%20WK2%20(Tests)/builds/5473/steps/run-api-tests/logs/stdio

Tim Horton

Comment 7 2019-09-07 15:38:16 PDT

Weird, I’ll take a peek

Tim Horton

Comment 8 2019-09-07 21:15:11 PDT

Oh, that assertion is very silly. I'm going to remove it.

Tim Horton

Comment 9 2019-09-07 21:18:15 PDT

Removed the assertion in http://trac.webkit.org/changeset/249624/webkit

Note You need to log in before you can comment on or make changes to this bug.