After working out some bridge improvements and fixing some Platform API bugs and testing, I believe it's ready. I'll start a [VOTE] thread soon.
On Thu, Dec 3, 2015 at 5:19 PM, Carlos Santana <csantan...@gmail.com> wrote: > +1 let's move forward this items with WKWebview I don't see holding up the > platform. > > Don't see any changes going on the platform, if any they will go into the > pluggable webview plugin or creating new plugin to handle. > On Wed, Dec 2, 2015 at 8:40 PM Shazron <shaz...@gmail.com> wrote: > >> Regarding Platform API, Vladimir Kotikov and Sergey Grebnov agree in >> the PR comments that the changes can go in to cordova-ios-4.x: >> https://github.com/apache/cordova-ios/pull/176 >> >> On Wed, Dec 2, 2015 at 5:38 PM, Shazron <shaz...@gmail.com> wrote: >> > Also, check footnote 3 above. >> > >> > Yes, WebKit defines window.openDatabase but it doesn't do anything. >> > Not sure why their tests didn't catch this... (see footnote for the >> > bug) >> > >> > With CSP off to rule things out: >> > XHR to yourself of course works, but doesn't really make sense for >> > real-world use. XHR to a sibling file, parent file, or any child file >> > results in the error ""Cross origin requests are only supported for >> > HTTP”. >> > >> > To illustrate: >> > >> > | >> > parent.xml >> > | >> > www >> > |---- index.html (file currently loaded) >> > |---- sibling.xml >> > |---- child-folder >> > | |---- child.xml >> > >> > index.html is the currently loaded file in the WebView. From it, you >> > can't load parent.xml, sibling.xml nor child.xml using XHR according >> > to my tests. >> > >> > Regarding *why* we have these storage tests, that is out of scope for >> > this discussion, but I agree. >> > >> > >> > On Wed, Dec 2, 2015 at 6:37 AM, Carlos Santana <csantan...@gmail.com> >> wrote: >> >> I'm guessing "pending" is the same as skipping the test. >> >> I'm guessing WKWebView doesn't support Web SQL, but window.openDatabase >> >> exist but it doesn't do anything? >> >> I ask because I only saw the pending for wkwebview spec.18 for using it, >> >> not for spec.9 where it checks that exists. >> >> Anyway after all questions, why the we are still testing for storage >> APIs? >> >> Cordova doesn't supported code to provide this storage APIs. >> >> I think we should remove the storage tests all together, this is >> >> webview/browser testing space. >> >> >> >> As for local xhr, is the problem only with specifying "../" relative >> path >> >> in the xhr url and not local resources? >> >> I see that doing xhr "index.html" that's a local resource and it works, >> and >> >> also "./" also passes. >> >> Aren't all this relative paths transform into full urls, and they will >> have >> >> file:// in the final path used? >> >> This means that xhr "folder1/data.json" works, but xhr >> >> "../someparent/data.json" doesn't? >> >> >> >> >> >> On Wed, Dec 2, 2015 at 4:52 AM Shazron <shaz...@gmail.com> wrote: >> >> >> >>> Marked the two known failures as pending. Now everything is green (and >> >>> yellow) across the board for UIWebView and WKWebView. >> >>> >> >>> On Tue, Dec 1, 2015 at 11:49 PM, Jesse <purplecabb...@gmail.com> >> wrote: >> >>> >>> Or should I just let it fail still? >> >>> > It depends how long it'll be until we fix them. The build will be >> broken >> >>> > in the CI until it is fixed so probably marking them as pending is >> the >> >>> > better option. >> >>> > >> >>> > >> >>> > @purplecabbage >> >>> > risingj.com >> >>> > >> >>> > On Tue, Dec 1, 2015 at 10:42 PM, Shazron <shaz...@gmail.com> wrote: >> >>> > >> >>> >> Couldn't wait. All file-transfer specs now pass for uiwebview and >> >>> >> wkwebview. >> >>> >> >> >>> >> For those two WKWebView tests that are failing, but are expected to >> >>> >> fail -- I'll try to modify the tests to mark the test as pending if >> >>> >> the platform is iOS and the WKWebView bridge is found. >> >>> >> >> >>> >> Or should I just let it fail still? >> >>> >> >> >>> >> On Tue, Dec 1, 2015 at 9:49 PM, Shazron <shaz...@gmail.com> wrote: >> >>> >> > Thanks! - yeah after I posted it, of course I realized it is all >> open >> >>> >> > source (duh) and I can run a local server or throw it on a >> >>> >> > digitalocean instance or something :) >> >>> >> > I'll do that tomorrow... >> >>> >> > >> >>> >> > On Tue, Dec 1, 2015 at 9:24 PM, Carlos Santana < >> csantan...@gmail.com> >> >>> >> wrote: >> >>> >> >> For a second I read "the bar is clear", but then I went to my >> fridge >> >>> and >> >>> >> >> saw I still have some beer left :-) >> >>> >> >> >> >>> >> >> How long before the INFRA provides the VM for the file transfer, >> I >> >>> >> looked >> >>> >> >> the JIRA and it mentioned something like "complete" and "we are >> in >> >>> >> holding >> >>> >> >> because of capacity" in the same comment, and I was like stupid >> >>> because >> >>> >> I >> >>> >> >> didn't understand :-( >> >>> >> >> >> >>> >> >> If it's going to take a long time, can we do the test with a >> local >> >>> >> machine >> >>> >> >> and vet that it works? >> >>> >> >> >> >>> >> >> For local xhr loading, I left a comment in the JIRA. I don't >> think >> >>> it's >> >>> >> >> need it but I'm curios on how local xhr loading works when >> fetching >> >>> >> normal >> >>> >> >> files on a web app, meaning dynamically loading js, css, html in >> SPA >> >>> >> using >> >>> >> >> a typical js framework like angular, etc.. >> >>> >> >> >> >>> >> >> Platform API is [5], I think is nice to have but not required >> for the >> >>> >> 4.0 >> >>> >> >> release. >> >>> >> >> >> >>> >> >> Oh by the way GREAT PROGRESS !!! and I cheers , I'm having a beer >> >>> now. >> >>> >> >> >> >>> >> >> >> >>> >> >> >> >>> >> >> On Tue, Dec 1, 2015 at 8:38 PM Shazron <shaz...@gmail.com> >> wrote: >> >>> >> >> >> >>> >> >>> The board is almost clear [1]. >> >>> >> >>> >> >>> >> >>> UIWebView mobile-spec passes, just waiting for INFRA-10831 [2] >> for >> >>> the >> >>> >> >>> file-transfer tests. >> >>> >> >>> Ditto for WKWebView, it essentially just fails two tests, which >> are >> >>> >> >>> expected [3] >> >>> >> >>> (filed a feature request issue [4] for local xhr loading, if >> >>> needed). >> >>> >> >>> >> >>> >> >>> Platform API [4] could go in this release as well, what do you >> >>> think? >> >>> >> >>> >> >>> >> >>> --- >> >>> >> >>> >> >>> >> >>> [1] >> >>> https://issues.apache.org/jira/secure/RapidBoard.jspa?rapidView=76 >> >>> >> >>> [2] https://issues.apache.org/jira/browse/INFRA-10831 >> >>> >> >>> [3] >> >>> >> >>> >> >>> >> >> >>> >> https://issues.apache.org/jira/browse/CB-7287?focusedCommentId=15034831&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15034831 >> >>> >> >>> [4] https://issues.apache.org/jira/browse/CB-10109 >> >>> >> >>> [5] https://github.com/apache/cordova-ios/pull/176 >> >>> >> >>> >> >>> >> >>> >> >>> --------------------------------------------------------------------- >> >>> >> >>> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org >> >>> >> >>> For additional commands, e-mail: dev-h...@cordova.apache.org >> >>> >> >>> >> >>> >> >>> >> >>> >> >> >>> >> >> --------------------------------------------------------------------- >> >>> >> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org >> >>> >> For additional commands, e-mail: dev-h...@cordova.apache.org >> >>> >> >> >>> >> >> >>> >> >>> --------------------------------------------------------------------- >> >>> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org >> >>> For additional commands, e-mail: dev-h...@cordova.apache.org >> >>> >> >>> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org >> For additional commands, e-mail: dev-h...@cordova.apache.org >> >> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
