@christianliebel 
> Does “shared cookie pool” mean that the IAB can share cookies with the 
> WebView as well? 

Yes, it uses the [CDVWKProcessPoolFactory 
class](https://github.com/apache/cordova-plugin-wkwebview-engine/blob/master/src/ios/CDVWKProcessPoolFactory.m)
 from 
[cordova-plugin-wkwebview-engine](https://github.com/apache/cordova-plugin-wkwebview-engine)
 in order to share cookies between the Cordova app Webview and the IAB Webview 
so setting a cookie in one makes it available in the other. This is currently 
enabled by default, but could be made optional.

> Unfortunately, there are new merge conflicts…

Since the master branch is being continually updated, this PR branch will 
undoubtedly get out of sync with it. But there's no point in me continually 
resolving merge conflicts if this PR is never going to be reviewed/merged.

@shazron If `cordova-plugin-wkwebview-engine` is integrated into 
`cordova-ios@5`, maybe it would make sense to align this PR with 
`cordova-ios@5`?

This would eliminate the need for the `__has_include` Objective-C macro which 
is used to detect the conditional presence of the 
`cordova-plugin-wkwebview-engine` classes in the build, but which only appears 
to work in the device environment so is currently causing the Travis CI build 
to fail due to tests failing in the Simulator.

[ Full content available at: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/271 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to