Bug 132405 – REGRESSION(r168041): http/tests/navigation/no-referrer-reset.html crashes on Mavericks WK1

• Status: RESOLVED
• Resolution: INVALID
• Component: History

[Brian Burg]

Fix forthcoming. Basically, I forgot to iron out/remove BackForwardClient::close(), which is no longer necessary since the client's lifetime is tied to the owning BackForwardController or WebBackForwardList.

[Brian Burg]

Created attachment 230531 [details]
the patch

[Andreas Kling]

Comment on attachment 230531 [details]
the patch

Regressions: Unexpected text-only failures (27)
  compositing/plugins/composited-plugin.html [ Failure ]
  compositing/plugins/no-backing-store.html [ Failure ]
  compositing/plugins/small-to-large-composited-plugin.html [ Failure ]
  fast/frames/sandboxed-iframe-about-blank.html [ Failure ]
  fast/frames/sandboxed-iframe-navigation-allowed.html [ Failure ]
  fast/frames/sandboxed-iframe-plugins.html [ Failure ]
  fast/replaced/invalid-object-with-fallback.html [ Failure ]
  http/tests/plugins/create-v8-script-objects.html [ Failure ]
  http/tests/plugins/cross-frame-object-access.html [ Failure ]
  http/tests/plugins/get-url-beforeunload-destroys-plugin.html [ Failure ]
  platform/mac/plugins/convert-point.html [ Failure ]
  plugins/destroy-during-npp-new.html [ Failure ]
  plugins/destroy-plugin-from-callback.html [ Failure ]
  plugins/npruntime/browser-object-identity.html [ Failure ]
  plugins/npruntime/construct.html [ Failure ]
  plugins/npruntime/delete-plugin-within-invoke.html [ Failure ]
  plugins/npruntime/embed-property-equality.html [ Failure ]
  plugins/npruntime/embed-property-iframe-equality.html [ Failure ]
  plugins/npruntime/embed-property.html [ Failure ]
  plugins/npruntime/enumerate.html [ Failure ]
  plugins/npruntime/evaluate.html [ Failure ]
  plugins/npruntime/get-int-identifier-special-values.html [ Failure ]
  plugins/npruntime/get-property-return-value.html [ Failure ]
  plugins/npruntime/identifier-conversion.html [ Failure ]
  plugins/npruntime/invoke-browserfuncs.html [ Failure ]
  plugins/npruntime/invoke-default.html [ Failure ]
  plugins/npruntime/invoke.html [ Failure ]

Regressions: Unexpected timeouts (3)
  fast/loader/image-in-page-cache.html [ Timeout ]
  http/tests/navigation/no-referrer-reset.html [ Timeout ]
  loader/load-defer-resume-crash.html [ Timeout ]

[Brian Burg]

Still a few things wrong.

WK1's WebBackForwardList is unconditionally freeing its backForwardList, but if the WebBackForwardList was initialized using initaliazeWithBackForwardList:backForwardList, then it doesn't own that list and shouldn't try to delete it.

[Brian Burg]

Created attachment 230550 [details]
the patch

[Brian Burg]

(In reply to comment #2)
> (From update of attachment 230531 [details])
> Regressions: Unexpected text-only failures (27)
>   compositing/plugins/composited-plugin.html [ Failure ]
>   compositing/plugins/no-backing-store.html [ Failure ]
>   compositing/plugins/small-to-large-composited-plugin.html [ Failure ]
>   fast/frames/sandboxed-iframe-about-blank.html [ Failure ]
>   fast/frames/sandboxed-iframe-navigation-allowed.html [ Failure ]
>   fast/frames/sandboxed-iframe-plugins.html [ Failure ]
>   fast/replaced/invalid-object-with-fallback.html [ Failure ]
>   http/tests/plugins/create-v8-script-objects.html [ Failure ]
>   http/tests/plugins/cross-frame-object-access.html [ Failure ]
>   http/tests/plugins/get-url-beforeunload-destroys-plugin.html [ Failure ]
>   platform/mac/plugins/convert-point.html [ Failure ]
>   plugins/destroy-during-npp-new.html [ Failure ]
>   plugins/destroy-plugin-from-callback.html [ Failure ]
>   plugins/npruntime/browser-object-identity.html [ Failure ]
>   plugins/npruntime/construct.html [ Failure ]
>   plugins/npruntime/delete-plugin-within-invoke.html [ Failure ]
>   plugins/npruntime/embed-property-equality.html [ Failure ]
>   plugins/npruntime/embed-property-iframe-equality.html [ Failure ]
>   plugins/npruntime/embed-property.html [ Failure ]
>   plugins/npruntime/enumerate.html [ Failure ]
>   plugins/npruntime/evaluate.html [ Failure ]
>   plugins/npruntime/get-int-identifier-special-values.html [ Failure ]
>   plugins/npruntime/get-property-return-value.html [ Failure ]
>   plugins/npruntime/identifier-conversion.html [ Failure ]
>   plugins/npruntime/invoke-browserfuncs.html [ Failure ]
>   plugins/npruntime/invoke-default.html [ Failure ]
>   plugins/npruntime/invoke.html [ Failure ]
> 
> Regressions: Unexpected timeouts (3)
>   fast/loader/image-in-page-cache.html [ Timeout ]
>   http/tests/navigation/no-referrer-reset.html [ Timeout ]
>   loader/load-defer-resume-crash.html [ Timeout ]

Plugin failures are probably from WebKitLegacy and related stormy weather today.

[Alexey Proskuryakov]

Resolved via rollout.