You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Shazron Abdullah (JIRA)" <ji...@apache.org> on 2015/11/26 23:09:11 UTC
[jira] [Comment Edited] (CB-7287) [WKWebView] Failing tests
[ 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: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org