[jira] [Commented] (CB-9615) getsettings API removed from Android CordovaWebView which disables renderpriority and WebView tweaks for devices below Android 5

2016-01-11 Thread Chris Murdaugh (JIRA)

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

Chris Murdaugh commented on CB-9615:


Thanks Nikhil and Joe.

I just learned today that as of Sep, 2015, Google increased the APK size limit 
on Google Play to 100mb (from 50mb). Joy!  So being able to incorporate 
Crosstalk 3rd party library is now an option for us (an probably most other 
developers).

Thanks to you both for your advice and attention. I have downgraded the issue 
to minor and agree it is resolved.

Thanks again.

Chris


> getsettings API removed from Android CordovaWebView which disables 
> renderpriority and WebView tweaks for devices below Android 5
> 
>
> Key: CB-9615
> URL: https://issues.apache.org/jira/browse/CB-9615
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Android
>Affects Versions: 4.1.0, 4.3.0, 4.0.1, 4.1.1
> Environment: Windows 8.1 and Android SDK with Eclipse or Android 
> Studio
>Reporter: Chris Murdaugh
>Priority: Minor
>
> getsettings API removed from CordovaWebView ex: 
> appView.getSettings().setRenderPriority in the java MainActivity no longer 
> works in Cordova Android 3.7.2 and above (to test you will need to set 
> hardwareacceleration to false in AndroidManifest and use the above java in 
> the mainactivity) -  an undefined method is thrown in the compiler. This is a 
> deal-breaker as we cannot use Crosswalk because it adds 18mb to APK size (we 
> really need to stay below the 50MB cellular APK size limit for market reach). 
> We need our CSS3 transforms to be above 1-10fps for stability and the above 
> tweak has allowed us to maintain compatibility and performance for Android 
> 4.x and above (this issue affects all supported Android versions since CSS3 
> transforms are slow without Crosswalk). Having the getsettings API removed 
> from Cordova 3.72 and above (and no replacement API exists) means that we 
> cannot upgrade our Cordova until this issue is resolved and will have to go 
> without patch for CVE-2015-1835.



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

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-9615) getsettings API removed from Android CordovaWebView which disables renderpriority and WebView tweaks for devices below Android 5

2016-01-11 Thread Nikhil Khandelwal (JIRA)

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

Nikhil Khandelwal commented on CB-9615:
---

Thanks, Chris for your comments and justification. I was not involved in the 
decision to remove this API but my understanding is that we removed it as we 
designed to support Chrome webview. I understand this an upgrade blocker for 
you but using Chrome webview is our recommendation to improve performance in 
such scenarios. Changing thread priority is not recommended and the API has 
also been deprecated in later versions of Android. [~bowserj] might have more 
background on this change and to see if there is a workaround available.

> getsettings API removed from Android CordovaWebView which disables 
> renderpriority and WebView tweaks for devices below Android 5
> 
>
> Key: CB-9615
> URL: https://issues.apache.org/jira/browse/CB-9615
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Android
>Affects Versions: 4.1.0, 4.3.0, 4.0.1, 4.1.1
> Environment: Windows 8.1 and Android SDK with Eclipse or Android 
> Studio
>Reporter: Chris Murdaugh
>
> getsettings API removed from CordovaWebView ex: 
> appView.getSettings().setRenderPriority in the java MainActivity no longer 
> works in Cordova Android 3.7.2 and above (to test you will need to set 
> hardwareacceleration to false in AndroidManifest and use the above java in 
> the mainactivity) -  an undefined method is thrown in the compiler. This is a 
> deal-breaker as we cannot use Crosswalk because it adds 18mb to APK size (we 
> really need to stay below the 50MB cellular APK size limit for market reach). 
> We need our CSS3 transforms to be above 1-10fps for stability and the above 
> tweak has allowed us to maintain compatibility and performance for Android 
> 4.x and above (this issue affects all supported Android versions since CSS3 
> transforms are slow without Crosswalk). Having the getsettings API removed 
> from Cordova 3.72 and above (and no replacement API exists) means that we 
> cannot upgrade our Cordova until this issue is resolved and will have to go 
> without patch for CVE-2015-1835.



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

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-9615) getsettings API removed from Android CordovaWebView which disables renderpriority and WebView tweaks for devices below Android 5

2016-01-11 Thread Joe Bowser (JIRA)

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

Joe Bowser commented on CB-9615:


getSettings() is not a Cordova API, and it was never guaranteed to work.  You 
can try and get the SystemWebView to get access to this API, but relying on 
non-Cordova APIs that aren't documented in the Cordova documentation is unwise.

> getsettings API removed from Android CordovaWebView which disables 
> renderpriority and WebView tweaks for devices below Android 5
> 
>
> Key: CB-9615
> URL: https://issues.apache.org/jira/browse/CB-9615
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Android
>Affects Versions: 4.1.0, 4.3.0, 4.0.1, 4.1.1
> Environment: Windows 8.1 and Android SDK with Eclipse or Android 
> Studio
>Reporter: Chris Murdaugh
>
> getsettings API removed from CordovaWebView ex: 
> appView.getSettings().setRenderPriority in the java MainActivity no longer 
> works in Cordova Android 3.7.2 and above (to test you will need to set 
> hardwareacceleration to false in AndroidManifest and use the above java in 
> the mainactivity) -  an undefined method is thrown in the compiler. This is a 
> deal-breaker as we cannot use Crosswalk because it adds 18mb to APK size (we 
> really need to stay below the 50MB cellular APK size limit for market reach). 
> We need our CSS3 transforms to be above 1-10fps for stability and the above 
> tweak has allowed us to maintain compatibility and performance for Android 
> 4.x and above (this issue affects all supported Android versions since CSS3 
> transforms are slow without Crosswalk). Having the getsettings API removed 
> from Cordova 3.72 and above (and no replacement API exists) means that we 
> cannot upgrade our Cordova until this issue is resolved and will have to go 
> without patch for CVE-2015-1835.



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

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-9615) getsettings API removed from Android CordovaWebView which disables renderpriority and WebView tweaks for devices below Android 5

2016-01-10 Thread Chris Murdaugh (JIRA)

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

Chris Murdaugh commented on CB-9615:


Hi Again Nikhil,

Just wanted to add one more comment..

Even 4.3 is slow as molasses (slower in some cases than 4.1 and 4.2) for CSS3 
animation frame rate - as you know, 4.3 is only a Chrome-based webview and 
Chromium Webview doesn't come in until kitkat 4.4 and above...

Even on 4.4 and above I have seen cases where setting the RenderPriority with 
getSettings can boost CSS tranform framerate...

The developer should be given the choice considering the totally anemic 
performance of Google's Webview for CSS (and Canvas) animation in its native 
webview...

I understand you are lightening the load of Cordova for performance. My 
suggestion would be rather than killing off the getSettings API for Webview, it 
could be moved into a plugin - So the developer can choose whether to adopt 3rd 
party library like crosstalk or opt to use getSettings to disable GPU rendering 
to boost framerate for applications requiring HTML/Canvas animation...

Thanks again.

Chris
 

> getsettings API removed from Android CordovaWebView which disables 
> renderpriority and WebView tweaks for devices below Android 5
> 
>
> Key: CB-9615
> URL: https://issues.apache.org/jira/browse/CB-9615
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Android
>Affects Versions: 4.1.0, 4.3.0, 4.0.1, 4.1.1
> Environment: Windows 8.1 and Android SDK with Eclipse or Android 
> Studio
>Reporter: Chris Murdaugh
>
> getsettings API removed from CordovaWebView ex: 
> appView.getSettings().setRenderPriority in the java MainActivity no longer 
> works in Cordova Android 3.7.2 and above (to test you will need to set 
> hardwareacceleration to false in AndroidManifest and use the above java in 
> the mainactivity) -  an undefined method is thrown in the compiler. This is a 
> deal-breaker as we cannot use Crosswalk because it adds 18mb to APK size (we 
> really need to stay below the 50MB cellular APK size limit for market reach). 
> We need our CSS3 transforms to be above 1-10fps for stability and the above 
> tweak has allowed us to maintain compatibility and performance for Android 
> 4.x and above (this issue affects all supported Android versions since CSS3 
> transforms are slow without Crosswalk). Having the getsettings API removed 
> from Cordova 3.72 and above (and no replacement API exists) means that we 
> cannot upgrade our Cordova until this issue is resolved and will have to go 
> without patch for CVE-2015-1835.



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

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-9615) getsettings API removed from Android CordovaWebView which disables renderpriority and WebView tweaks for devices below Android 5

2016-01-09 Thread Chris Murdaugh (JIRA)

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

Chris Murdaugh commented on CB-9615:


The problem is two-fold:

1. As Android 4.3 and below are the old webview (not Chrome-based webview), CSS 
Tranforms are slow. This kills our app and many others which use CSS tranforms 
for some important HTML animations. LOTS of people are complaining about this 
and it has been a longstanding issue for Android performance for 
Cordova/Phonegap:

For evidence of the large number of developers this issue impacts, google: 
"cordova css transforms slow"
But to save you time, just read this one (has a long chain of many developers 
discussing the issue):
https://forum.ionicframework.com/t/slow-page-transitions-on-android-with-hardware-acceleration-on/799/18


2. Google is limiting APK size to 50MB (for cellular download), which for most 
developers on Cordova/Phonegap that means most of the market and the Cordova 
APK expansion plugins aren't really there yet...

3. Crosswalk consumes around 25mb extra file size to include the library with 
Cordova to address the CSS Tranforms issue.. This kills it for most developers 
(see point 2). Androids 4.0 - 4.3 are still 27.4% of Android device 
distribution - as per:
http://developer.android.com/about/dashboards/index.html

4. The decision to pull support for the old getsettings API in this light is 
unwise and I can promise you will turn potential developers away from 
Cordova/Phonegap (considering that Android is over 80% of the mobile market and 
until 4.3 and below are well and truly extinct this will plague Cordova 
uptake)...

Please consider adding a shim or some sort of call to allow 
.getSettings().setRenderPriorityto work..



> getsettings API removed from Android CordovaWebView which disables 
> renderpriority and WebView tweaks for devices below Android 5
> 
>
> Key: CB-9615
> URL: https://issues.apache.org/jira/browse/CB-9615
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Android
>Affects Versions: 4.1.0, 4.3.0, 4.0.1, 4.1.1
> Environment: Windows 8.1 and Android SDK with Eclipse or Android 
> Studio
>Reporter: Chris Murdaugh
>Priority: Minor
>
> getsettings API removed from CordovaWebView ex: 
> appView.getSettings().setRenderPriority in the java MainActivity no longer 
> works in Cordova Android 3.7.2 and above (to test you will need to set 
> hardwareacceleration to false in AndroidManifest and use the above java in 
> the mainactivity) -  an undefined method is thrown in the compiler. This is a 
> deal-breaker as we cannot use Crosswalk because it adds 18mb to APK size (we 
> really need to stay below the 50MB cellular APK size limit for market reach). 
> We need our CSS3 transforms to be above 1-10fps for stability and the above 
> tweak has allowed us to maintain compatibility and performance for Android 
> 4.x and above (this issue affects all supported Android versions since CSS3 
> transforms are slow without Crosswalk). Having the getsettings API removed 
> from Cordova 3.72 and above (and no replacement API exists) means that we 
> cannot upgrade our Cordova until this issue is resolved and will have to go 
> without patch for CVE-2015-1835.



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

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-9615) getsettings API removed from Android CordovaWebView which disables renderpriority and WebView tweaks for devices below Android 5

2016-01-08 Thread Nikhil Khandelwal (JIRA)

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

Nikhil Khandelwal commented on CB-9615:
---

Can you provide why you need this API? It's unlikely we will resurrect this API.

> getsettings API removed from Android CordovaWebView which disables 
> renderpriority and WebView tweaks for devices below Android 5
> 
>
> Key: CB-9615
> URL: https://issues.apache.org/jira/browse/CB-9615
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Android
>Affects Versions: 4.1.0, 4.3.0, 4.0.1, 4.1.1
> Environment: Windows 8.1 and Android SDK with Eclipse or Android 
> Studio
>Reporter: Chris Murdaugh
>Priority: Minor
>
> getsettings API removed from CordovaWebView ex: 
> appView.getSettings().setRenderPriority in the java MainActivity no longer 
> works in Cordova Android 3.7.2 and above (to test you will need to set 
> hardwareacceleration to false in AndroidManifest and use the above java in 
> the mainactivity) -  an undefined method is thrown in the compiler. This is a 
> deal-breaker as we cannot use Crosswalk because it adds 18mb to APK size (we 
> really need to stay below the 50MB cellular APK size limit for market reach). 
> We need our CSS3 transforms to be above 1-10fps for stability and the above 
> tweak has allowed us to maintain compatibility and performance for Android 
> 4.x and above (this issue affects all supported Android versions since CSS3 
> transforms are slow without Crosswalk). Having the getsettings API removed 
> from Cordova 3.72 and above (and no replacement API exists) means that we 
> cannot upgrade our Cordova until this issue is resolved and will have to go 
> without patch for CVE-2015-1835.



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

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org