From 177746d05e1ceefe8421a4fdab7407e1d17b46d6 Mon Sep 17 00:00:00 2001
From: John Reck <jreck@google.com>
Date: Thu, 20 Jan 2011 16:59:23 -0800
Subject: Remove the bookmark star from history context menu

 Bug: 3374340
 The star isn't supposed to be in the context menu, removed

Change-Id: I2dbd59ca32f7bec66468adf8b9c76ba1a425a94a
---
 src/com/android/browser/HistoryItem.java | 46 +++++++++++++++++++-------------
 1 file changed, 28 insertions(+), 18 deletions(-)

(limited to 'src/com/android/browser/HistoryItem.java')

diff --git a/src/com/android/browser/HistoryItem.java b/src/com/android/browser/HistoryItem.java
index ccedc016b..8154be79f 100644
--- a/src/com/android/browser/HistoryItem.java
+++ b/src/com/android/browser/HistoryItem.java
@@ -21,37 +21,33 @@ import android.content.Context;
 import android.provider.Browser;
 import android.view.View;
 import android.widget.CompoundButton;
+import android.widget.CompoundButton.OnCheckedChangeListener;
 
 /**
  *  Layout representing a history item in the classic history viewer.
  */
-/* package */ class HistoryItem extends BookmarkItem {
+/* package */ class HistoryItem extends BookmarkItem
+        implements OnCheckedChangeListener {
 
     private CompoundButton  mStar;      // Star for bookmarking
-    private CompoundButton.OnCheckedChangeListener  mListener;
     /**
      *  Create a new HistoryItem.
      *  @param context  Context for this HistoryItem.
      */
     /* package */ HistoryItem(Context context) {
+        this(context, true);
+    }
+
+    /* package */ HistoryItem(Context context, boolean showStar) {
         super(context);
 
         mStar = (CompoundButton) findViewById(R.id.star);
-        mStar.setVisibility(View.VISIBLE);
-        mListener = new CompoundButton.OnCheckedChangeListener() {
-            public void onCheckedChanged(CompoundButton buttonView,
-                    boolean isChecked) {
-                if (isChecked) {
-                    // Uncheck ourseves. When the bookmark is actually added,
-                    // we will be notified
-                    setIsBookmark(false);
-                    Browser.saveBookmark(getContext(), getName(), mUrl);
-                } else {
-                    Bookmarks.removeFromBookmarks(getContext(),
-                            getContext().getContentResolver(), mUrl, getName());
-                }
-            }
-        };
+        mStar.setOnCheckedChangeListener(this);
+        if (showStar) {
+            mStar.setVisibility(View.VISIBLE);
+        } else {
+            mStar.setVisibility(View.GONE);
+        }
     }
     
     /* package */ void copyTo(HistoryItem item) {
@@ -75,6 +71,20 @@ import android.widget.CompoundButton;
     /* package */ void setIsBookmark(boolean isBookmark) {
         mStar.setOnCheckedChangeListener(null);
         mStar.setChecked(isBookmark);
-        mStar.setOnCheckedChangeListener(mListener);
+        mStar.setOnCheckedChangeListener(this);
+    }
+
+    @Override
+    public void onCheckedChanged(CompoundButton buttonView,
+            boolean isChecked) {
+        if (isChecked) {
+            // Uncheck ourseves. When the bookmark is actually added,
+            // we will be notified
+            setIsBookmark(false);
+            Browser.saveBookmark(getContext(), getName(), mUrl);
+        } else {
+            Bookmarks.removeFromBookmarks(getContext(),
+                    getContext().getContentResolver(), mUrl, getName());
+        }
     }
 }
-- 
cgit v1.2.3