r228299 broke reader mode in Safari.
<rdar://problem/37399012>
Created attachment 333600 [details] Patch
Can an API test (or something) be made to test this?
(In reply to Sam Weinig from comment #3) > Can an API test (or something) be made to test this? I do not understand the issue yet so I haven't been able to write a test. It seems to be some kind of race as reader mode sometimes works and sometimes doesn't.
Created attachment 333617 [details] Rebased
(In reply to Chris Dumez from comment #4) > (In reply to Sam Weinig from comment #3) > > Can an API test (or something) be made to test this? > > I do not understand the issue yet so I haven't been able to write a test. It > seems to be some kind of race as reader mode sometimes works and sometimes > doesn't. I believe I now understand the issue and I am working on writing an API test. This said, this may take some time.
Created attachment 333643 [details] Patch
Comment on attachment 333643 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=333643&action=review > Source/WebCore/ChangeLog:26 > + riskier change at this point. I will follow-up on this issue. I filed https://bugs.webkit.org/show_bug.cgi?id=182720 for this.
(In reply to Chris Dumez from comment #0) > r228299 broke reader mode in Safari. Can you please elaborate?
(In reply to Daniel Bates from comment #9) > (In reply to Chris Dumez from comment #0) > > r228299 broke reader mode in Safari. > > Can you please elaborate? Never mind. You described how it broke here: > Source/WebCore/ChangeLog:22 > + the policy response from the client. When the provisional load is committed, > + we call FrameLoader::stopLoading() which after r228299 cancelled pending > + policy checks. Because we did not wait for the policy check response to > + commit the load, we would cancel it which would make the load fail.
Comment on attachment 333643 [details] Patch Clearing flags on attachment: 333643 Committed r228430: <https://trac.webkit.org/changeset/228430>
All reviewed patches have been landed. Closing bug.