commit 70a153b61fc7606e6bb4dd279013ba87804b86e6
author Leon Scroggins <scroggo@google.com> Mon May 10 17:27:26 2010 -0400
committer Leon Scroggins <scroggo@google.com> Tue May 11 08:49:03 2010 -0400
tree 4d8336bd3fb4d2372b48499f6a1035328d25291a
parent 79e36d97b67bf06b1e0c0edf773882e93a3293e4

Add debugging information to help fix bug 2289137.

Change-Id: I0ef5d006988267ac8dd5f2f6d42c923fea90ecc4

src/com/android/browser/Tab.java

diff --git a/src/com/android/browser/Tab.java b/src/com/android/browser/Tab.java
index 921ecb3..b583dbc 100644
--- a/src/com/android/browser/Tab.java
+++ b/src/com/android/browser/Tab.java
@@ -1825,6 +1825,9 @@
         // FIXME: The only place we cared about subwindow was for
         // bookmarking (i.e. not when saving state). Was this deliberate?
         final WebBackForwardList list = mMainView.copyBackForwardList();
+        if (list == null) {
+            Log.w(LOGTAG, "populatePickerData called and WebBackForwardList is null");
+        }
         final WebHistoryItem item = list != null ? list.getCurrentItem() : null;
         populatePickerData(item);
     }
@@ -1833,7 +1836,9 @@
     // WebView.
     private void populatePickerData(WebHistoryItem item) {
         mPickerData = new PickerData();
-        if (item != null) {
+        if (item == null) {
+            Log.w(LOGTAG, "populatePickerData called with a null WebHistoryItem");
+        } else {
             mPickerData.mUrl = item.getUrl();
             mPickerData.mTitle = item.getTitle();
             mPickerData.mFavicon = item.getFavicon();