Hi Raphael,
See my comments below. I'll paste an updated intent after resolve all your concern. -----Original Message----- From: Crosswalk-dev [mailto:[email protected]] On Behalf Of [email protected] Sent: Tuesday, March 31, 2015 4:47 PM To: [email protected] Subject: Crosswalk-dev Digest, Vol 19, Issue 56 Send Crosswalk-dev mailing list submissions to [email protected]<mailto:[email protected]> To subscribe or unsubscribe via the World Wide Web, visit https://lists.crosswalk-project.org/mailman/listinfo/crosswalk-dev or, via email, send a message with subject or body 'help' to [email protected]<mailto:[email protected]> You can reach the person managing the list at [email protected]<mailto:[email protected]> When replying, please edit your Subject line so it is more specific than "Re: Contents of Crosswalk-dev digest..." Today's Topics: 1. Re: Intent to Implement: LZMA support for Crosswalk (Raphael Kubo da Costa) 2. Crosswalk Nightly Automation Test Report on 13.42.323.0 (Zhang, XiaoyuX) ---------------------------------------------------------------------- Message: 1 Date: Tue, 31 Mar 2015 11:06:33 +0300 From: Raphael Kubo da Costa <[email protected]<mailto:[email protected]>> To: [email protected]<mailto:[email protected]> Subject: Re: [Crosswalk-dev] Intent to Implement: LZMA support for Crosswalk Message-ID: <[email protected]<mailto:[email protected]>> Content-Type: text/plain; charset=utf-8 Some of the questions I asked below still have not been answered as far as I can tell. Raphael Kubo da Costa <[email protected]<mailto:[email protected]>> writes: > "Dong, Jun" <[email protected]<mailto:[email protected]>> writes: > >> This use the LZMA library from SevenZip >> (http://sourceforge.net/projects/sevenzip/files/). > > What's the license of this code? Where will it be located? > LZMA SDK is written and placed in the public domain by Igor Pavlov. The license shipped with the LZMA SDK package(in lzma.txt), you can download here: http://www.7-zip.org/sdk.html >> The app will firstly extract the compressed library in its package, >> if it doesn?t found the library or the library is older. > > Can you elaborate? So the idea is to compress libxwalkcore.so, > decompress it once when loading and store the uncompressed version > somewhere and start using that one? > Yes, app will check /data/data/your_app/app_xwalkcore/libxwalkcore.so first, if it's not exists or older than the compressed one(in apk), app will do the decompressing work after it launched. >> Crosswalk will provide a new method(onXWalkReady) for developers to >> customize their behavior when the decompression is completed. > > What kind of customization do you have in mind? At first it sounds > unnecessary to me. > You can only create XWalkView(and some related classes) at onXWalkReady, because they need to wait libxwalkcore.so ready. >> Will add a new build flag(use_lzma), by default it is disabled in >> Crosswalk. > > I guess this is related to me not understanding the exact thing you > are proposing here, but why do we want a build flag if this offers > clear benefits? Are there any drawbacks you did not mention? > No, just decompressing, this will take several seconds. Besides, there's another option for developers who care the size most(Crosswalk Lite), for Crosswalk we can't assume that and let developers to balance the size & launch time issue. >> Risks/Opens: >> >> 1. Rebasing may introduce some regression. > > Why? Does this introduce changes to chromium-crosswalk? No. 1. LZMA support is firstly completed on Crosswalk Lite, since about 4 month ago, so there're a lot of code conflict I need to resolve. 2. I suppose some test case will fail, because this feature change the app's behavior(need to wait at first time), and I've notice the QA in advance. ------------------------------ Message: 2 Date: Tue, 31 Mar 2015 08:46:40 +0000 From: "Zhang, XiaoyuX" <[email protected]<mailto:[email protected]>> To: "'[email protected]'" <[email protected]<mailto:[email protected]>>, "'[email protected]'" <[email protected]<mailto:[email protected]>> Cc: "Li, JunX B" <[email protected]<mailto:[email protected]>>, "Jiang, WenhaoX" <[email protected]<mailto:[email protected]>>, "Dai, FengX" <[email protected]<mailto:[email protected]>>, "Hao, YunfeiX" <[email protected]<mailto:[email protected]>> Subject: [Crosswalk-dev] Crosswalk Nightly Automation Test Report on 13.42.323.0 Message-ID: <031aba5a0f2bee459e8a09f5f218b8ff78e...@shsmsx102.ccr.corp.intel.com<mailto:031aba5a0f2bee459e8a09f5f218b8ff78e...@shsmsx102.ccr.corp.intel.com>> Content-Type: text/plain; charset="us-ascii" Test Objective: Capture the key regression of daily build by automated nightly testing. Summary: : One major issue found after comparing recent result, total pass rate is decreasing compared with 13.42.322.0. Aging downgrade issue(since 12.41.296.0):,XWALK-3527<https://crosswalk-project.org/jira/browse/XWALK-3527>, XWALK-3529<https://crosswalk-project.org/jira/browse/XWALK-3529>, XWALK-3532<https://crosswalk-project.org/jira/browse/XWALK-3532> Aging downgrade issue(since 13.42.319.0):XWALK-3794<https://crosswalk-project.org/jira/browse/XWALK-3794> XWALK-3795<https://crosswalk-project.org/jira/browse/XWALK-3795> XWALK-3797<https://crosswalk-project.org/jira/browse/XWALK-3797> Aging downgrade issue(since 13.42.321.0):XWALK-3816<https://crosswalk-project.org/jira/browse/XWALK-3816> XWALK-3812<https://crosswalk-project.org/jira/browse/XWALK-3812> Downgraded Components: Platform Test component TC Num Pass% Diff w/ 13.41.318.0 Comments Android x86/arm Crosswalk Use Cases/WebAPI sample 35 91% -9% XWALK-3844<https://crosswalk-project.org/jira/browse/XWALK-3844> Overall Test Results: Release Variant Branch Hardware WebAPI Runtime Features Embedding APIs Use Cases SampleApp Diff w/ 13.42.321.0 Total TCs Pass% Trend Total TCs Pass% Trend Total TCs Pass% Trend Total TCs Pass% Trend Total TCs Pass% Trend Crosswalk Embeded Mode Canary ASUS MeMO Pad 8 (4.4.2) 11838 89% -- 630 100% -- 198 89% -- 35 91% -9% 47 100% -- bug XWALK-3816 block webapi test on ARM device and bug XWALK-3812 block cordova webapi test . ZTE Geek V975 (4.2.2) 11492 88% -2% 630 100% -- 198 89% -- 35 91% -9% 42 100% -- Toshiba Excite Go AT7-C8(4.4.2) 11492 88% -2% 630 100% -- 198 89% -- 35 91% -9% 42 100% -- Google Nexus 4(4.2.2) N/A N/A N/A 630 100% -- 198 89% -- 35 91% -9% 47 100% -- Google Nexus 7 (4.4.4) N/A N/A N/A 630 100% -- 198 89% -- 35 91% -9% 42 100% -- Crosswalk-Cordova Full Canary ASUS MeMO Pad 8 (4.4.2) N/A N/A N/A N/A N/A N/A N/A N/A N/A 35 91% -9% 47 100% -- ZTE Geek V975 (4.2.2) N/A N/A N/A N/A N/A N/A N/A N/A N/A 35 91% -9% 42 100% -- Toshiba Excite Go AT7-C8(4.4.2) N/A N/A N/A N/A N/A N/A N/A N/A N/A 35 91% -9% 42 100% -- Google Nexus 4(4.2.2) N/A N/A N/A N/A N/A N/A N/A N/A N/A 35 91% -9% 47 100% -- Google Nexus 7 (4.4.4) N/A N/A N/A N/A N/A N/A N/A N/A N/A 35 91% -9% 42 100% -- Test Coverage: Full Auto TCs of Web APIs, Crosswalk WRT, Cordova WebAPIs, Embedding APIs, Use Case and SampleApp Test Environment: Crosswalk build: https://download.01.org/crosswalk/releases/crosswalk/ Tizen-Extensions-Crosswalk: https://download.01.org/crosswalk/releases/tizen-extensions-crosswalk/ Github Test Suite Links: https://github.com/crosswalk-project/crosswalk-test-suite/ Thanks, Crosswalk QA team -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.crosswalk-project.org/pipermail/crosswalk-dev/attachments/20150331/ac6ba77c/attachment.html> ------------------------------ Subject: Digest Footer _______________________________________________ Crosswalk-dev mailing list [email protected]<mailto:[email protected]> https://lists.crosswalk-project.org/mailman/listinfo/crosswalk-dev ------------------------------ End of Crosswalk-dev Digest, Vol 19, Issue 56 *********************************************
_______________________________________________ Crosswalk-dev mailing list [email protected] https://lists.crosswalk-project.org/mailman/listinfo/crosswalk-dev
