 Chromium Code Reviews
 Chromium Code Reviews Issue 2691373004:
  [Android History] Even more polish  (Closed)
    
  
    Issue 2691373004:
  [Android History] Even more polish  (Closed) 
  | Index: chrome/android/java/src/org/chromium/chrome/browser/history/HistoryAdapter.java | 
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/history/HistoryAdapter.java b/chrome/android/java/src/org/chromium/chrome/browser/history/HistoryAdapter.java | 
| index 824d8964b51bca918ed72fa3761b66d543a55d10..799784d5fa6028968b86e7f2d668a7a2cdfa4266 100644 | 
| --- a/chrome/android/java/src/org/chromium/chrome/browser/history/HistoryAdapter.java | 
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/history/HistoryAdapter.java | 
| @@ -52,6 +52,7 @@ public class HistoryAdapter extends DateDividedAdapter implements BrowsingHistor | 
| private final ArrayList<HistoryItemView> mItemViews; | 
| private RecyclerView mRecyclerView; | 
| + private ViewGroup mPrivacyDisclaimers; | 
| private TextView mSignedInNotSyncedTextView; | 
| private TextView mSignedInSyncedTextView; | 
| private TextView mOtherFormsOfBrowsingHistoryTextView; | 
| @@ -313,6 +314,8 @@ public class HistoryAdapter extends DateDividedAdapter implements BrowsingHistor | 
| mHistoryManager.getDefaultLateralListItemMarginPx(), 0); | 
| updateClearBrowsingDataButtonVisibility(); | 
| + mPrivacyDisclaimers = (ViewGroup) v.findViewById(R.id.privacy_disclaimers); | 
| + | 
| mSignedInNotSyncedTextView = (TextView) v.findViewById(R.id.signed_in_not_synced); | 
| setPrivacyDisclaimerText(mSignedInNotSyncedTextView, | 
| R.string.android_history_no_synced_results, LEARN_MORE_LINK); | 
| @@ -380,6 +383,10 @@ public class HistoryAdapter extends DateDividedAdapter implements BrowsingHistor | 
| mSignedInSyncedTextView.setVisibility(mHasSyncedData ? View.VISIBLE : View.GONE); | 
| mOtherFormsOfBrowsingHistoryTextView.setVisibility( | 
| mHasOtherFormsOfBrowsingData ? View.VISIBLE : View.GONE); | 
| + | 
| + boolean privacyDisclaimersVisible = | 
| 
gone
2017/02/16 19:29:16
arePrivacyDisclaimersVisible?  This function alrea
 
Theresa
2017/02/16 19:59:10
Done.
 | 
| + isSignedIn || mHasSyncedData || mHasOtherFormsOfBrowsingData; | 
| + mPrivacyDisclaimers.setVisibility(privacyDisclaimersVisible ? View.VISIBLE : View.GONE); | 
| } | 
| private void updateClearBrowsingDataButtonVisibility() { |