[ 
https://issues.apache.org/jira/browse/CB-7287?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15029277#comment-15029277
 ] 

Shazron Abdullah edited comment on CB-7287 at 11/26/15 10:08 PM:
-----------------------------------------------------------------

Found the culprit for the infinite "gap://ready" calls when the WKWebView 
engine plugin is installed for mobile-spec.
During runtime, checking cordova.exec shows the right clobber, but the old 
iOSExec call is called at least once by cordova.js, resulting in this infinite 
call.

pokeNative() is called infinitely here: 
https://github.com/apache/cordova-ios/blob/e3ae7e3ca0395988a087884f9ba18a683160dbde/cordova-js-src/exec.js#L162

Not sure if this can be handled in JavaScript, it seems that wkwebview-engine 
has to handle this to satisfy the old exec at least once. In any case, the old 
iOSExec function design is inadequate. It is more desirable to handle this in 
JavaScript since the wkwebview-engine ideally does not need to know about how 
gap://ready works.

cc [~purplecabbage] who broke out the .js exec functionality




was (Author: shazron):
Found the culprit for the infinite "gap://ready" calls when the WKWebView 
engine plugin is installed for mobile-spec.
During runtime, checking cordova.exec shows the right clobber, but the iOSExec 
call is called at least once by cordova.js, resulting in this infinite call.

pokeNative() is called infinitely here: 
https://github.com/apache/cordova-ios/blob/e3ae7e3ca0395988a087884f9ba18a683160dbde/cordova-js-src/exec.js#L162

Not sure if this can be handled in JavaScript, it seems that wkwebview-engine 
has to handle this to satisfy the old exec at least once. In any case, the old 
iOSExec function design is inadequate. It is more desirable to handle this in 
JavaScript since the wkwebview-engine ideally does not need to know about how 
gap://ready works.

cc [~purplecabbage] who broke out the .js exec functionality



> [WKWebView] Failing tests
> -------------------------
>
>                 Key: CB-7287
>                 URL: https://issues.apache.org/jira/browse/CB-7287
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: iOS
>         Environment: iOS 8
>            Reporter: Shazron Abdullah
>            Assignee: Anis Kadri
>              Labels: cordova-ios-4.0.x
>
> This is only for the wkwebview branch, and using these plugins, and 3.5.0 
> mobile-spec:
> {code}
>     "org.apache.cordova.battery-status": "0.2.10",
>     "org.apache.cordova.camera": "0.3.1",
>     "org.apache.cordova.console": "0.2.10",
>     "org.apache.cordova.contacts": "0.2.12",
>     "org.apache.cordova.device": "0.2.11",
>     "org.apache.cordova.device-motion": "0.2.9",
>     "org.apache.cordova.device-orientation": "0.3.8",
>     "org.apache.cordova.dialogs": "0.2.9",
>     "org.apache.cordova.file": "1.3.0",
>     "org.apache.cordova.file-transfer": "0.4.5",
>     "org.apache.cordova.geolocation": "0.3.9",
>     "org.apache.cordova.globalization": "0.3.0",
>     "org.apache.cordova.inappbrowser": "0.5.1",
>     "org.apache.cordova.media": "0.2.12",
>     "org.apache.cordova.media-capture": "0.3.2",
>     "org.apache.cordova.network-information": "0.2.11",
>     "org.apache.cordova.splashscreen": "0.3.2",
>     "org.apache.cordova.vibration": "0.3.10"
> {code}
> The 3 failing tests:
> {code}
> FileTransfer upload method filetransfer.spec.18 should be able to upload a 
> file.
> Expected 
> '{"fields":{"value2":"param","value1":"test"},"files":{"file":{"size":24,"path":"/opt/run/snapshot/.tmp/ed60ec667664b454d72b193bf8841e5c","name":"upload.txt","type":"text/plain","mtime":"2014-08-12T21:53:28.862Z"}}}
>  ' to match /fields:\s*{\s*value1.*/.
> FileTransfer upload method filetransfer.spec.19 should be able to upload a 
> file with http basic auth.
> Expected 
> '{"fields":{"value2":"param","value1":"test"},"files":{"file":{"size":24,"path":"/opt/run/snapshot/.tmp/94aa407aca7bd9c53081f0da5ab33d54","name":"upload.txt","type":"text/plain","mtime":"2014-08-12T21:53:29.322Z"}}}
>  ' to match /fields:\s*{\s*value1.*/.
> FileTransfer Backwards compatibility filetransfer.spec.29 should be able to 
> upload a file using local paths.
> Expected 
> '{"fields":{"value2":"param","value1":"test"},"files":{"file":{"size":24,"path":"/opt/run/snapshot/.tmp/9bf857a3d64ef9beaf4eae4e22876ecc","name":"upload.txt","type":"text/plain","mtime":"2014-08-12T21:53:31.307Z"}}}
>  ' to match /fields:\s*{\s*value1.*/.
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to