public/cordova-android
public/cordova-android
9
0
Fork 0
forked from github/cordova-android
Code Pull Requests Activity

Fix for Console.log, didn't call the super of the method needed

Browse Source
This commit is contained in:
Joe Bowser 2011-10-25 15:21:06 -07:00
parent aa6e011255
commit 80ee464dce
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
framework/src/com/phonegap/DroidGap.java
View File

@ -1027,6 +1027,9 @@ public class DroidGap extends PhonegapActivity {
* If the client returns true, WebView will assume that the client will * If the client returns true, WebView will assume that the client will
* handle the prompt dialog and call the appropriate JsPromptResult method. * handle the prompt dialog and call the appropriate JsPromptResult method.
* *
* Since we are hacking prompts for our own purposes, we should not be using them for
* this purpose, perhaps we should hack console.log to do this instead!
*
* @param view * @param view
* @param url * @param url
* @param message * @param message
@ -1158,13 +1161,14 @@ public class DroidGap extends PhonegapActivity {
public void onConsoleMessage(String message, int lineNumber, String sourceID) public void onConsoleMessage(String message, int lineNumber, String sourceID)
{ {
LOG.d(TAG, "%s: Line %d : %s", sourceID, lineNumber, message); LOG.d(TAG, "%s: Line %d : %s", sourceID, lineNumber, message);
super.onConsoleMessage(message, lineNumber, sourceID);
} }
@Override @Override
public boolean onConsoleMessage(ConsoleMessage consoleMessage) public boolean onConsoleMessage(ConsoleMessage consoleMessage)
{ {
LOG.d(TAG, consoleMessage.message()); LOG.d(TAG, consoleMessage.message());
return true; return super.onConsoleMessage(consoleMessage);
} }
@Override @Override