Bug 205028 – [WK2][iOS] Visibility propagation view is not properly updated when navigating cross-process

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit2

[Chris Dumez]

Visibility propagation view is not properly updated when navigating cross-process on iOS WK2.

[Chris Dumez]

Created attachment 385193 [details]
Patch

[Chris Dumez]

Created attachment 385198 [details]
Patch

[Tim Horton]

Comment on attachment 385198 [details]
Patch

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

> Source/WebKit/UIProcess/SuspendedPageProxy.h:50
> +using LayerHostingContextID = uint32_t;

Odd that we have this in so many places

[Chris Dumez]

Comment on attachment 385198 [details]
Patch

Clearing flags on attachment: 385198

Committed r253337: <https://trac.webkit.org/changeset/253337>

[Chris Dumez]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/57803133>

[Truitt Savell]

It looks like the changes in https://trac.webkit.org/changeset/253337/webkit

caused testing to exit early on Mac Debug wk2

Build:
https://build.webkit.org/builders/Apple%20Mojave%20Debug%20WK2%20%28Tests%29/builds/6312

I was able to reproduce this on 253337 but not 253336 by just running the http/tests/ directory.

[Chris Dumez]

(In reply to Truitt Savell from comment #7)
> It looks like the changes in https://trac.webkit.org/changeset/253337/webkit
> 
> caused testing to exit early on Mac Debug wk2
> 
> Build:
> https://build.webkit.org/builders/Apple%20Mojave%20Debug%20WK2%20%28Tests%29/
> builds/6312
> 
> I was able to reproduce this on 253337 but not 253336 by just running the
> http/tests/ directory.

Follow-up in <https://trac.webkit.org/changeset/253354>.