Hey

Can you file a bug in JIRA against this commit so we can track it?  I
didn't realize that this would break anything significant.  I've
already modified it so that it should be fixed.  The problem that I
have with this is that now that we have two places to check for the
proper WebViewClient instead of one.  The bug that this is fixing is
the issue with our WebHistory.  CB-1568.

Joe

On Thu, Oct 25, 2012 at 1:01 PM, Bryce Curtis <[email protected]> wrote:
> Joe,
>
> The commit
>
> https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=commitdiff;h=6aafd6dc3aec1ed1fe1d7a3e08d73deedddbc3a3
>
> changes the signature of a public method
>
> -    public void init(CordovaWebView webView, CordovaWebViewClient
> webViewClient, CordovaChromeClient webChromeClient) {
> +    public void init(CordovaWebView webView, CordovaChromeClient
> webChromeClient) {
>
> which breaks any app that extends DroidGap & calls init with their own
> view and chrome clients.
>
> We really need this support, so could you please revert.
>
> What is the bug that this was addressing - and maybe there's another
> way to solve it?  Let me know if Simon or I can help.
>
> Thanks,
> Bryce

Reply via email to