Re: [Android] Deprecate HTC 2.3 Android console.log support in 3.0

2013-05-13 Thread Jesse
console.log is pretty useful.
This technically isn't our API, but a polyfill for platforms where
console.log goes nowhere.
WP7+8 have this broken into the DebugConsole plugin, and  clobber
window.console as well as navigator.console.

modulemapper.clobbers('cordova/plugin/windowsphone/console',
'navigator.console');
modulemapper.clobbers('cordova/plugin/windowsphone/console', 'console');

The actual implementation is bare bones[1], and does not have all the
log-level bs in other versions.


[1]
https://github.com/apache/cordova-js/blob/master/lib/windowsphone/plugin/windowsphone/console.js


@purplecabbage
risingj.com


On Mon, May 13, 2013 at 2:22 PM, Joe Bowser bows...@gmail.com wrote:

 Hey

 There's a known bug in HTC's WebView that suppressed console.log on
 old Gingerbread devices.  This is only on devices that run HTC Sense,
 and not devices like the Nexus One.  How do people feel about
 deprecating console.log in 3.0.  I know that Simon hated this
 particular chunk of code, and wanted to toss it by the wayside a while
 ago.

 It says it was supposed to be ripped out this release, but I don't see
 it on the wiki, so I'm giving it until 3.0 to be removed.  Does that
 sound reasonable?

 Joe



Re: [Android] Deprecate HTC 2.3 Android console.log support in 3.0

2013-05-13 Thread Andrew Grieve
Could you be more specific about what you'd like to get rid of?

Logger plugin?
onConsoleMessage?


On Mon, May 13, 2013 at 5:47 PM, Jesse purplecabb...@gmail.com wrote:

 console.log is pretty useful.
 This technically isn't our API, but a polyfill for platforms where
 console.log goes nowhere.
 WP7+8 have this broken into the DebugConsole plugin, and  clobber
 window.console as well as navigator.console.

 modulemapper.clobbers('cordova/plugin/windowsphone/console',
 'navigator.console');
 modulemapper.clobbers('cordova/plugin/windowsphone/console', 'console');

 The actual implementation is bare bones[1], and does not have all the
 log-level bs in other versions.


 [1]

 https://github.com/apache/cordova-js/blob/master/lib/windowsphone/plugin/windowsphone/console.js


 @purplecabbage
 risingj.com


 On Mon, May 13, 2013 at 2:22 PM, Joe Bowser bows...@gmail.com wrote:

  Hey
 
  There's a known bug in HTC's WebView that suppressed console.log on
  old Gingerbread devices.  This is only on devices that run HTC Sense,
  and not devices like the Nexus One.  How do people feel about
  deprecating console.log in 3.0.  I know that Simon hated this
  particular chunk of code, and wanted to toss it by the wayside a while
  ago.
 
  It says it was supposed to be ripped out this release, but I don't see
  it on the wiki, so I'm giving it until 3.0 to be removed.  Does that
  sound reasonable?
 
  Joe
 



Re: [Android] Deprecate HTC 2.3 Android console.log support in 3.0

2013-05-13 Thread Simon MacDonald
I believe Joe means this bit of code:

// Set the nav dump for HTC 2.x devices (disabling for ICS,
deprecated entirely for Jellybean 4.2)

try {

Method gingerbread_getMethod =  WebSettings.class.getMethod(
setNavDump, new Class[] { boolean.class });



String manufacturer = android.os.Build.MANUFACTURER;

Log.d(TAG, CordovaWebView is running on device made by:  +
manufacturer);

if(android.os.Build.VERSION.SDK_INT  android.os.Build.
VERSION_CODES.HONEYCOMB 

android.os.Build.MANUFACTURER.contains(HTC))

{

gingerbread_getMethod.invoke(settings, true);

}

} catch (NoSuchMethodException e) {

Log.d(TAG, We are on a modern version of Android, we will
deprecate HTC 2.3 devices in 2.8);

} catch (IllegalArgumentException e) {

Log.d(TAG, Doing the NavDump failed with bad arguments);

} catch (IllegalAccessException e) {

Log.d(TAG, This should never happen: IllegalAccessException
means this isn't Android anymore);

} catch (InvocationTargetException e) {

Log.d(TAG, This should never happen: InvocationTargetException
means this isn't Android anymore.);

}

I'm in favour of getting rid of it if that is indeed what Joe is
suggestion.

Simon Mac Donald
http://hi.im/simonmacdonald


On Mon, May 13, 2013 at 8:43 PM, Andrew Grieve agri...@chromium.org wrote:

 Could you be more specific about what you'd like to get rid of?

 Logger plugin?
 onConsoleMessage?


 On Mon, May 13, 2013 at 5:47 PM, Jesse purplecabb...@gmail.com wrote:

  console.log is pretty useful.
  This technically isn't our API, but a polyfill for platforms where
  console.log goes nowhere.
  WP7+8 have this broken into the DebugConsole plugin, and  clobber
  window.console as well as navigator.console.
 
  modulemapper.clobbers('cordova/plugin/windowsphone/console',
  'navigator.console');
  modulemapper.clobbers('cordova/plugin/windowsphone/console', 'console');
 
  The actual implementation is bare bones[1], and does not have all the
  log-level bs in other versions.
 
 
  [1]
 
 
 https://github.com/apache/cordova-js/blob/master/lib/windowsphone/plugin/windowsphone/console.js
 
 
  @purplecabbage
  risingj.com
 
 
  On Mon, May 13, 2013 at 2:22 PM, Joe Bowser bows...@gmail.com wrote:
 
   Hey
  
   There's a known bug in HTC's WebView that suppressed console.log on
   old Gingerbread devices.  This is only on devices that run HTC Sense,
   and not devices like the Nexus One.  How do people feel about
   deprecating console.log in 3.0.  I know that Simon hated this
   particular chunk of code, and wanted to toss it by the wayside a while
   ago.
  
   It says it was supposed to be ripped out this release, but I don't see
   it on the wiki, so I'm giving it until 3.0 to be removed.  Does that
   sound reasonable?
  
   Joe
  
 



Re: [Android] Deprecate HTC 2.3 Android console.log support in 3.0

2013-05-13 Thread Joe Bowser
Yes, I want to get rid of that pretty bad piece of Java awfulness.

On Mon, May 13, 2013 at 5:55 PM, Simon MacDonald
simon.macdon...@gmail.com wrote:
 I believe Joe means this bit of code:

 // Set the nav dump for HTC 2.x devices (disabling for ICS,
 deprecated entirely for Jellybean 4.2)

 try {

 Method gingerbread_getMethod =  WebSettings.class.getMethod(
 setNavDump, new Class[] { boolean.class });



 String manufacturer = android.os.Build.MANUFACTURER;

 Log.d(TAG, CordovaWebView is running on device made by:  +
 manufacturer);

 if(android.os.Build.VERSION.SDK_INT  android.os.Build.
 VERSION_CODES.HONEYCOMB 

 android.os.Build.MANUFACTURER.contains(HTC))

 {

 gingerbread_getMethod.invoke(settings, true);

 }

 } catch (NoSuchMethodException e) {

 Log.d(TAG, We are on a modern version of Android, we will
 deprecate HTC 2.3 devices in 2.8);

 } catch (IllegalArgumentException e) {

 Log.d(TAG, Doing the NavDump failed with bad arguments);

 } catch (IllegalAccessException e) {

 Log.d(TAG, This should never happen: IllegalAccessException
 means this isn't Android anymore);

 } catch (InvocationTargetException e) {

 Log.d(TAG, This should never happen: InvocationTargetException
 means this isn't Android anymore.);

 }

 I'm in favour of getting rid of it if that is indeed what Joe is
 suggestion.

 Simon Mac Donald
 http://hi.im/simonmacdonald


 On Mon, May 13, 2013 at 8:43 PM, Andrew Grieve agri...@chromium.org wrote:

 Could you be more specific about what you'd like to get rid of?

 Logger plugin?
 onConsoleMessage?


 On Mon, May 13, 2013 at 5:47 PM, Jesse purplecabb...@gmail.com wrote:

  console.log is pretty useful.
  This technically isn't our API, but a polyfill for platforms where
  console.log goes nowhere.
  WP7+8 have this broken into the DebugConsole plugin, and  clobber
  window.console as well as navigator.console.
 
  modulemapper.clobbers('cordova/plugin/windowsphone/console',
  'navigator.console');
  modulemapper.clobbers('cordova/plugin/windowsphone/console', 'console');
 
  The actual implementation is bare bones[1], and does not have all the
  log-level bs in other versions.
 
 
  [1]
 
 
 https://github.com/apache/cordova-js/blob/master/lib/windowsphone/plugin/windowsphone/console.js
 
 
  @purplecabbage
  risingj.com
 
 
  On Mon, May 13, 2013 at 2:22 PM, Joe Bowser bows...@gmail.com wrote:
 
   Hey
  
   There's a known bug in HTC's WebView that suppressed console.log on
   old Gingerbread devices.  This is only on devices that run HTC Sense,
   and not devices like the Nexus One.  How do people feel about
   deprecating console.log in 3.0.  I know that Simon hated this
   particular chunk of code, and wanted to toss it by the wayside a while
   ago.
  
   It says it was supposed to be ripped out this release, but I don't see
   it on the wiki, so I'm giving it until 3.0 to be removed.  Does that
   sound reasonable?
  
   Joe