[Home] Add BottomSheetObserverTest#testSheetContentChanged

chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetContentController.java

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.widget.bottomsheet;
 
 import android.content.Context;
 import android.content.res.Resources;
 import android.support.annotation.IntDef;
 import android.support.design.internal.BottomNavigationItemView;
 import android.support.design.internal.BottomNavigationMenuView;
 import android.support.design.widget.BottomNavigationView;
 import android.support.design.widget.BottomNavigationView.OnNavigationItemSelectedListener;
 import android.util.AttributeSet;
 import android.view.MenuItem;
 import android.view.View;
 
+import org.chromium.base.VisibleForTesting;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.bookmarks.BookmarkSheetContent;
 import org.chromium.chrome.browser.download.DownloadSheetContent;
 import org.chromium.chrome.browser.history.HistorySheetContent;
 import org.chromium.chrome.browser.suggestions.SuggestionsBottomSheetContent;
 import org.chromium.chrome.browser.tabmodel.TabModelSelector;
 import org.chromium.chrome.browser.util.MathUtils;
 import org.chromium.chrome.browser.widget.bottomsheet.BottomSheet.BottomSheetContent;
 
 import java.lang.annotation.Retention;
@@ skipping 149 matching lines @@
     private void showBottomSheetContent(int navItemId) {
         // There are some bugs related to programatically selecting menu items that are fixed in
         // newer support library versions.
         // TODO(twellington): remove this after the support library is rolled.
         if (mSelectedItemId != 0) getMenu().findItem(mSelectedItemId).setChecked(false);
         mSelectedItemId = navItemId;
         getMenu().findItem(mSelectedItemId).setChecked(true);
 
         mBottomSheet.showContent(getSheetContentForId(mSelectedItemId));
     }
+
+    /**
+     * @param itemId The id of the MenuItem to select.
+     */
+    @VisibleForTesting
+    public void selectItem(int itemId) {

[gone, 2017/03/21 17:46:51]

If this is only meant for testing, call it that so no one else calls it.

selectItemForTests

[Theresa, 2017/03/21 22:29:24]

Done.

+        // TODO(twellington): A #setSelectedItemId() method was added to the support library
+        //                    recently. Replace this custom implementation with that method after
+        //                    the support library is rolled.
+        onNavigationItemSelected(getMenu().findItem(itemId));
+    }
 }