Bug 177825 – Add ObjC SPI equivalent to WKPageLoaderClient.willGoToBackForwardListItem

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Alex Christensen]

Add ObjC SPI equivalent to WKPageLoaderClient.willGoToBackForwardListItem

[Alex Christensen]

Created attachment 322551 [details]
Patch

[Tim Horton]

Comment on attachment 322551 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=322551&action=review

> Source/WebKit/UIProcess/WebPageProxy.cpp:1213
> +        if (m_navigationClient->canHandleWillGoToBackForwardListItem())
> +            m_navigationClient->willGoToBackForwardListItem(*this, *item, inPageCache, m_process->transformHandlesToObjects(userData.object()).get());

In both of these patches: didn't we previously use a bool return value to specify whether it was handled, instead of an extra function? I don't care either way, but it would be good to be consistent unless you have reason not to.

[Alex Christensen]

Created attachment 322552 [details]
Patch

[Alex Christensen]

http://trac.webkit.org/r222794

[Radar WebKit Bug Importer]

<rdar://problem/34795612>

[Ryan Haddad]

This change caused 11 API tests crashes on macOS:

Tests that failed:
  DeviceScaleFactorOnBack.WebKit2
  WebKit.MouseMoveAfterCrash
  WebKit.RestoreSessionStateContainingFormData
  WebKit.RestoreSessionStateContainingScrollRestorationDefault
  WebKit.RestoreStateAfterTermination
  WebKit.ShouldGoToBackForwardListItem
  WebKit.ShouldKeepCurrentBackForwardListItemInList
  WebKit.WKPageCopySessionStateWithFiltering
  WebKit.WKPageGetScaleFactorNotZero
  WebKit2_CommandBackForwardTestWKView.LTR
  WebKit2_CommandBackForwardTestWKView.RTL

https://build.webkit.org/builders/Apple%20Sierra%20Release%20WK1%20%28Tests%29/builds/5020

They seem to be crashing in WebKit::WebPageProxy::willGoToBackForwardListItem(unsigned long long, bool, WebKit::UserData const&) + 87 (WebPageProxy.cpp:1212)

[Ryan Haddad]

Created attachment 322575 [details]
API test crashlog