Bug 955870 - Cache the array length in IsRecentlyVisitedURI r=blassey

This commit is contained in:
Mark Finkle 2014-01-02 14:36:06 -05:00
parent a860814a6c
commit ba6c1b4316
2 changed files with 7 additions and 6 deletions

View File

@ -128,7 +128,7 @@ public class BrowserToolbar extends GeckoRelativeLayout
private ImageButton mSiteSecurity;
private PageActionLayout mPageActionLayout;
private Animation mProgressSpinner;
//private Animation mProgressSpinner;
private TabCounter mTabsCounter;
private GeckoImageButton mMenu;
private GeckoImageView mMenuIcon;
@ -303,7 +303,7 @@ public class BrowserToolbar extends GeckoRelativeLayout
mSiteIdentityPopup = new SiteIdentityPopup(mActivity);
mSiteIdentityPopup.setAnchor(mSiteSecurity);
mProgressSpinner = AnimationUtils.loadAnimation(mActivity, R.anim.progress_spinner);
//mProgressSpinner = AnimationUtils.loadAnimation(mActivity, R.anim.progress_spinner);
mStop = (ImageButton) findViewById(R.id.stop);
mPageActionLayout = (PageActionLayout) findViewById(R.id.page_action_layout);
@ -785,8 +785,8 @@ public class BrowserToolbar extends GeckoRelativeLayout
// To stop the glitch caused by multiple start() calls.
if (!mSpinnerVisible) {
setPageActionVisibility(true);
mFavicon.setAnimation(mProgressSpinner);
mProgressSpinner.start();
//mFavicon.setAnimation(mProgressSpinner);
//mProgressSpinner.start();
mSpinnerVisible = true;
}
Log.i(LOGTAG, "zerdatime " + SystemClock.uptimeMillis() + " - Throbber start");
@ -799,7 +799,7 @@ public class BrowserToolbar extends GeckoRelativeLayout
if (mSpinnerVisible) {
setPageActionVisibility(false);
mFavicon.setAnimation(null);
mProgressSpinner.cancel();
//mProgressSpinner.cancel();
mSpinnerVisible = false;
}
Log.i(LOGTAG, "zerdatime " + SystemClock.uptimeMillis() + " - Throbber stop");

View File

@ -107,7 +107,8 @@ inline bool
nsAndroidHistory::IsRecentlyVisitedURI(nsIURI* aURI) {
bool equals = false;
RecentlyVisitedArray::index_type i;
for (i = 0; i < mRecentlyVisitedURIs.Length() && !equals; ++i) {
RecentlyVisitedArray::size_type length = mRecentlyVisitedURIs.Length();
for (i = 0; i < length && !equals; ++i) {
aURI->Equals(mRecentlyVisitedURIs.ElementAt(i), &equals);
}
return equals;