Bug 190801 – Don't waste time under -setupInteraction under -initWithFrame for unparented WKWebViews

• Component: New Bugs
• Status: RESOLVED FIXED

[Tim Horton]

Don't waste time under -setupInteraction under -initWithFrame for unparented WKWebViews

[Tim Horton]

Created attachment 352902 [details]
Patch

[Tim Horton]

<rdar://problem/43674361>

[Megan Gardner]

Comment on attachment 352902 [details]
Patch

Assuming all bots pass

[WebKit Commit Bot]

Comment on attachment 352902 [details]
Patch

Clearing flags on attachment: 352902

Committed r237331: <https://trac.webkit.org/changeset/237331>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Ryan Haddad]

(In reply to WebKit Commit Bot from comment #4)
> Committed r237331: <https://trac.webkit.org/changeset/237331>

This appears to have caused API test failures on iOS:

Failed

    TestWebKitAPI.WebKit.InteractionDeadlockAfterCrash
        
        /Volumes/Data/slave/ios-simulator-12-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/InteractionDeadlockAfterCrash.mm:70
        Value of: highlightLongPressRecognizer
          Actual: false
        Expected: true
        
        
        /Volumes/Data/slave/ios-simulator-12-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/InteractionDeadlockAfterCrash.mm:74
        Value of: shouldBegin
          Actual: false
        Expected: true
        

Timeout

    TestWebKitAPI.ActionSheetTests.DismissingActionSheetShouldNotDismissPresentingViewController

https://build.webkit.org/builders/Apple%20iOS%2012%20Simulator%20Release%20WK2%20(Tests)/builds/491/steps/run-api-tests/logs/stdio

[Tim Horton]

Technically a good change. I’ll fix it (by parenting the view).

[Tim Horton]

(In reply to Tim Horton from comment #7)
> Technically a good change. I’ll fix it (by parenting the view).

Ryan, https://trac.webkit.org/changeset/237346/webkit should do the trick.

[Ryan Haddad]

(In reply to Tim Horton from comment #8)
> (In reply to Tim Horton from comment #7)
> > Technically a good change. I’ll fix it (by parenting the view).
> 
> Ryan, https://trac.webkit.org/changeset/237346/webkit should do the trick.
That did indeed! We are still seeing TestWebKitAPI.ActionSheetTests.DismissingActionSheetShouldNotDismissPresentingViewController time out, though. Sorry for not making it clear that both were caused by this change.

[Tim Horton]

(In reply to Ryan Haddad from comment #9)
> (In reply to Tim Horton from comment #8)
> > (In reply to Tim Horton from comment #7)
> > > Technically a good change. I’ll fix it (by parenting the view).
> > 
> > Ryan, https://trac.webkit.org/changeset/237346/webkit should do the trick.
> That did indeed! We are still seeing
> TestWebKitAPI.ActionSheetTests.
> DismissingActionSheetShouldNotDismissPresentingViewController time out,
> though. Sorry for not making it clear that both were caused by this change.

Interesting! I'll take a peek.

[Tim Horton]

(In reply to Tim Horton from comment #10)
> (In reply to Ryan Haddad from comment #9)
> > (In reply to Tim Horton from comment #8)
> > > (In reply to Tim Horton from comment #7)
> > > > Technically a good change. I’ll fix it (by parenting the view).
> > > 
> > > Ryan, https://trac.webkit.org/changeset/237346/webkit should do the trick.
> > That did indeed! We are still seeing
> > TestWebKitAPI.ActionSheetTests.
> > DismissingActionSheetShouldNotDismissPresentingViewController time out,
> > though. Sorry for not making it clear that both were caused by this change.
> 
> Interesting! I'll take a peek.

Very similar problem, the view isn't in-window. Not sure yet exactly why it's not, since from code inspection it looks like it should be, but it's possible we just have to wait for a transition.