Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Date: Tue, 11 Sep 2012 14:30:08 +1100 (NCT)
From: "enzi.wang (JIRA)" <jira@apache.org>
To: callback-dev@incubator.apache.org
Message-ID: <1708403616.61539.1347334208615.JavaMail.jiratomcat@arcas>
In-Reply-To: <940445662.61530.1347334088941.JavaMail.jiratomcat@arcas>
Subject: [jira] [Updated] (CB-1408) DroidGap only clear  appView's history
  in clearHistory  method
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable


     [ https://issues.apache.org/jira/browse/CB-1408?page=3Dcom.atlassian.j=
ira.plugin.system.issuetabpanels:all-tabpanel ]

enzi.wang updated CB-1408:
--------------------------

    Description:=20
DroidGap only clear appView's history in clearHistory method.

DroidGap.java:
public void clearHistory() {
        this.appView.clearHistory();
}


But it also used managed urls in backHistory method:

public boolean backHistory() {
        if (this.appView !=3D null) {
            return appView.backHistory();
        }
        return false;
}

In CordovaWebView.java:

backHistory(){
  ...
  // If our managed history has prev url
        if (this.urls.size() > 1) {
            this.urls.pop();                // Pop current url
            String url =3D this.urls.pop();   // Pop prev url that we want =
to load, since it will be added back by loadUrl()
            this.loadUrl(url);
            return true;
        }
  ...
}

The managed urls is private.

We do not have any places to clear all of these histories:
1=E3=80=81appView's History
2=E3=80=81our managed urls history

  was:
DroidGap only clear appView's history in clearHistory method.

DroidGap.java:
/**
     * Clear web history in this web view.
     */
    public void clearHistory() {
        this.appView.clearHistory();
}


But it also used managed urls in backHistory method:

public boolean backHistory() {
        if (this.appView !=3D null) {
            return appView.backHistory();
        }
        return false;
}

In CordovaWebView.java:

backHistory(){
  ...
  // If our managed history has prev url
        if (this.urls.size() > 1) {
            this.urls.pop();                // Pop current url
            String url =3D this.urls.pop();   // Pop prev url that we want =
to load, since it will be added back by loadUrl()
            this.loadUrl(url);
            return true;
        }
  ...
}

The managed urls is private.

We do not have any places to clear all of these histories:
1=E3=80=81appView's History
2=E3=80=81our managed urls history

   =20
> DroidGap only clear  appView's history  in clearHistory  method
> ---------------------------------------------------------------
>
>                 Key: CB-1408
>                 URL: https://issues.apache.org/jira/browse/CB-1408
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Android
>    Affects Versions: 2.0.0
>            Reporter: enzi.wang
>            Assignee: Joe Bowser
>            Priority: Minor
>             Fix For: 2.1.0
>
>
> DroidGap only clear appView's history in clearHistory method.
> DroidGap.java:
> public void clearHistory() {
>         this.appView.clearHistory();
> }
> But it also used managed urls in backHistory method:
> public boolean backHistory() {
>         if (this.appView !=3D null) {
>             return appView.backHistory();
>         }
>         return false;
> }
> In CordovaWebView.java:
> backHistory(){
>   ...
>   // If our managed history has prev url
>         if (this.urls.size() > 1) {
>             this.urls.pop();                // Pop current url
>             String url =3D this.urls.pop();   // Pop prev url that we wan=
t to load, since it will be added back by loadUrl()
>             this.loadUrl(url);
>             return true;
>         }
>   ...
> }
> The managed urls is private.
> We do not have any places to clear all of these histories:
> 1=E3=80=81appView's History
> 2=E3=80=81our managed urls history

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrato=
rs
For more information on JIRA, see: http://www.atlassian.com/software/jira