[Signin][Android] Change account picker methods to package private

This CL changes several account picker methods to package private as
we already finished modularization of account picker.

Bug: 1155123
Change-Id: I33252c72ac37eaa121e35aea59578390018571ec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631499Reviewed-by: Tanmoy Mollik <triploblastic@chromium.org>
Commit-Queue: Alice Wang <aliceywang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844741}

chrome/browser/signin/ui/android/java/src/org/chromium/chrome/browser/signin/ui/account_picker/ExistingAccountRowViewBinder.java

@@ -22,9 +22,8 @@ import org.chromium.ui.modelutil.PropertyModelChangeProcessor.ViewBinder;
 /**
  * This class regroups the buildView and bindView util methods of the
  * existing account row.
- * TODO(https://crbug.com/1155123): Change this class to package internal after modularization
  */
-public class ExistingAccountRowViewBinder implements ViewBinder<PropertyModel, View, PropertyKey> {
+class ExistingAccountRowViewBinder implements ViewBinder<PropertyModel, View, PropertyKey> {
     /**
      * View binder that associates an existing account view with the model of
      * {@link ExistingAccountRowProperties}.
@@ -60,9 +59,8 @@ public class ExistingAccountRowViewBinder implements ViewBinder<PropertyModel, V
      *
      * @param profileData profile data needs to bind.
      * @param view A view object inflated from @layout/account_picker_row.
-     * TODO(https://crbug.com/1155123): Change this method to package internal after modularization
      */
-    public static void bindAccountView(DisplayableProfileData profileData, View view) {
+    static void bindAccountView(DisplayableProfileData profileData, View view) {
         ImageView accountImage = view.findViewById(R.id.account_image);
         accountImage.setImageDrawable(profileData.getImage());
 

chrome/browser/signin/ui/android/java/src/org/chromium/chrome/browser/signin/ui/account_picker/OnClickListenerViewBinder.java

@@ -14,16 +14,14 @@ import org.chromium.ui.modelutil.PropertyModelChangeProcessor.ViewBinder;
 
 /**
  * This class binds an {@link OnClickListener} to a {@link View}.
- * TODO(crbug/1155123): Change this class to package internal after modularization
  */
-public class OnClickListenerViewBinder implements ViewBinder<PropertyModel, View, PropertyKey> {
+class OnClickListenerViewBinder implements ViewBinder<PropertyModel, View, PropertyKey> {
     private final ReadableObjectPropertyKey<OnClickListener> mOnClickListenerKey;
 
     /**
-     * TODO(crbug/1155123): Change this method to package internal after modularization
+     * Constructs a view binder for a key of {@link OnClickListener}.
      */
-    public OnClickListenerViewBinder(
-            ReadableObjectPropertyKey<OnClickListener> onClickListenerKey) {
+    OnClickListenerViewBinder(ReadableObjectPropertyKey<OnClickListener> onClickListenerKey) {
         mOnClickListenerKey = onClickListenerKey;
     }