Return-Path: X-Original-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id D80259C1C for ; Thu, 29 Mar 2012 04:09:46 +0000 (UTC) Received: (qmail 43224 invoked by uid 500); 29 Mar 2012 04:09:46 -0000 Delivered-To: apmail-incubator-callback-dev-archive@incubator.apache.org Received: (qmail 43033 invoked by uid 500); 29 Mar 2012 04:09:46 -0000 Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: callback-dev@incubator.apache.org Delivered-To: mailing list callback-dev@incubator.apache.org Received: (qmail 43003 invoked by uid 99); 29 Mar 2012 04:09:45 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 29 Mar 2012 04:09:45 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of curtis.bryce@gmail.com designates 209.85.220.175 as permitted sender) Received: from [209.85.220.175] (HELO mail-vx0-f175.google.com) (209.85.220.175) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 29 Mar 2012 04:09:38 +0000 Received: by vcbfl13 with SMTP id fl13so1411296vcb.6 for ; Wed, 28 Mar 2012 21:09:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=Bw1Rb63i3oVOWCcwZ84AH4qFyYJcFqx3qj5Nic8hSC0=; b=zjI0vnsnQLAAfKOAJYEMghItwNC2Yxz2Qhb+Gb9IEFNU1Q7dfDfCS92z6v/f2VIzH4 Wr6pvBteAkUBYfTo/h2LqRGmBdRk/rQThXjdHzNvJIFvlkarieNLII2WiO+SXqyWWSSH faPU9tViBY9U+HV6BGVF2RS9eJCM9WE7WMZUNeIKjKXggLXqQ19qv+LrWC4MjbHFlCnp /RrEttdu0GjfpcjKhLT4Ilgf4j133zD4GGXjxGEdSXQn5zqvdaReJMkuY9IcCSt0uoIu YLB4AzmvOn6RBOaYzwShrpJ8l9MrVM99qxBfkWtGDl6QOl38oGaVoD0WurGnoZiCzANX CzQg== MIME-Version: 1.0 Received: by 10.52.28.200 with SMTP id d8mr12746842vdh.38.1332994158113; Wed, 28 Mar 2012 21:09:18 -0700 (PDT) Received: by 10.220.229.71 with HTTP; Wed, 28 Mar 2012 21:09:18 -0700 (PDT) In-Reply-To: References: Date: Wed, 28 Mar 2012 23:09:18 -0500 Message-ID: Subject: Re: [Android] CordovaWebView: Moving the CallbackServer and PluginManager out of DroidGap From: Bryce Curtis To: callback-dev@incubator.apache.org Content-Type: multipart/alternative; boundary=20cf3079bc9e48318404bc59e32b X-Virus-Checked: Checked by ClamAV on apache.org --20cf3079bc9e48318404bc59e32b Content-Type: text/plain; charset=ISO-8859-1 I really haven't had time to look at this in detail, but agree that anything related to the webview should be in CordovaWebView. As Fil mentioned, that includes the history, plugin manager, whitelisting, & authentication + callback server. I assume that overriding chrome/view clients so the user can specify their own will still work. On Wed, Mar 28, 2012 at 6:17 PM, Filip Maj wrote: > Sorry for late reply Joe! > > Looks great! As for outstanding issues as per your wiki article [1], I > would say move everything WebView related, as well as Cordova-specific > such as the plugin manager, into CordovaWebView.java. My thinking here is > that, none of scaffolding necessary to enable device APIs in the web view > should be a burden on the user - the CordovaWebView class should handle > all of that. > > It separates the cordova-y bits as something the WEbView needs to manage > on its own, as well, and cleans up the final Activity-extending class to > be simpler. Our end users should not have to worry about that stuff, nor > do they need to see it in their own activities, or the generated > activities the baseline tooling within cordova-android provides. > > IMO: history, plugin manager, whitelisting, authentication, should all be > handled by CordovaWebView. > > [1] http://wiki.apache.org/cordova/CordovaWebView > > On 3/28/12 4:06 PM, "Joe Bowser" wrote: > > >BUMP! Are we all on board with doing this? > > > >Joe > > > >On Tue, Mar 27, 2012 at 1:15 PM, Joe Bowser wrote: > > > >> Hey > >> > >> I've been working on the CordovaWebView branch, and I think we need to > >> discuss where to put the CallbackServer and PluginManager in the new > >> implementation. I'm OK with it being in the view, but I did have it in > >>the > >> Client before, and I'm wondering what people's thoughts are on that. > >>Also, > >> since these are core pieces of Cordova on Android, this may break the > >> branch, which is fine, but it'd be good if more people looked at this > >> branch, and discussed how this should work. > >> > >> > >> > >> > https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a > >>=shortlog;h=refs/heads/CordovaWebView > >> > >> http://wiki.apache.org/cordova/CordovaWebView > >> > >> Joe > >> > > --20cf3079bc9e48318404bc59e32b--