[Home] Record some user actions for the Chrome Home BottomSheet

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.design.internal.BottomNavigationItemView;
 import android.support.design.internal.BottomNavigationMenuView;
@@ skipping 105 matching lines @@
 
         showBottomSheetContent(R.id.action_home);
     }
 
     @Override
     public void onLoadUrl(String url) {}
 
     @Override
     public void onSheetOffsetChanged(float heightFraction) {}
 
+    @Override
+    public void onSheetStateChanged(int newState) {}
+
+    @Override
+    public void onSheetContentChanged(BottomSheetContent newContent) {}

[Theresa, 2017/03/17 16:07:12]

I'm considering making a new class variable for the BottomSheetObserver that
extends EmptyBottomSheetObserver to avoid the need for these empty methods.
Thoughts? Should I do the same for BottomToolbarPhone?

[mdjones, 2017/03/17 18:17:26]

Yeah, I think that interface is sufficiently large to support doing this.

+
     // TODO(twellington): remove this once the support library is updated to allow disabling
     //                    shifting mode or determines shifting mode based on the width of the
     //                    child views.
     private void disableShiftingMode() {
         BottomNavigationMenuView menuView = (BottomNavigationMenuView) getChildAt(0);
         try {
             Field shiftingMode = menuView.getClass().getDeclaredField("mShiftingMode");
             shiftingMode.setAccessible(true);
             shiftingMode.setBoolean(menuView, false);
             shiftingMode.setAccessible(false);
@@ skipping 32 matching lines @@
         // 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));
     }
 }