weblayer: minor rename

RemoteFragmentClient -> BrowserFragment

BUG=none
TEST=none

Change-Id: I7560db41694f766b71bf13c9f6f687d587b93b6d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1825569Reviewed-by: Pavel Shmakov <pshmakov@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699878}

weblayer/public/java/BUILD.gn

@@ -21,8 +21,8 @@ android_resources("client_resources") {
 
 android_library("java") {
   java_files = [
-    "org/chromium/weblayer/RemoteFragmentClient.java",
     "org/chromium/weblayer/BrowserController.java",
+    "org/chromium/weblayer/BrowserFragment.java",
     "org/chromium/weblayer/BrowserFragmentController.java",
     "org/chromium/weblayer/BrowserObserver.java",
     "org/chromium/weblayer/Callback.java",

weblayer/public/java/org/chromium/weblayer/RemoteFragmentClient.java → weblayer/public/java/org/chromium/weblayer/BrowserFragment.java

@@ -19,74 +19,68 @@ import org.chromium.weblayer_private.aidl.IRemoteFragmentClient;
 import org.chromium.weblayer_private.aidl.ObjectWrapper;
 
 /**
- * Hosts a "remote fragment" (represented by {@link IRemoteFragment}) that lives in another
- * ClassLoader. The remote fragment has all the actual logic (e.g. handling lifecycle events), while
- * this class actually extends {@link Fragment}, forwarding the calls to and from the remote
- * fragment. Thus it is "hosting" the fragment implemented elsewhere.
+ * WebLayer's fragment implementation.
  */
-public final class RemoteFragmentClient extends Fragment {
+public final class BrowserFragment extends Fragment {
     private final IRemoteFragmentClient mClientImpl = new IRemoteFragmentClient.Stub() {
         @Override
         public void superOnCreate(IObjectWrapper savedInstanceState) {
-            RemoteFragmentClient.super.onCreate(ObjectWrapper.unwrap(savedInstanceState,
-                    Bundle.class));
+            BrowserFragment.super.onCreate(ObjectWrapper.unwrap(savedInstanceState, Bundle.class));
         }
 
         @Override
         public void superOnAttach(IObjectWrapper context) {
-            RemoteFragmentClient.super.onAttach(ObjectWrapper.unwrap(context, Context.class));
+            BrowserFragment.super.onAttach(ObjectWrapper.unwrap(context, Context.class));
         }
 
         @Override
         public void superOnActivityCreated(IObjectWrapper savedInstanceState) {
-            RemoteFragmentClient.super.onCreate(ObjectWrapper.unwrap(savedInstanceState,
-                    Bundle.class));
+            BrowserFragment.super.onCreate(ObjectWrapper.unwrap(savedInstanceState, Bundle.class));
         }
 
         @Override
         public void superOnStart() {
-            RemoteFragmentClient.super.onStart();
+            BrowserFragment.super.onStart();
         }
 
         @Override
         public void superOnResume() {
-            RemoteFragmentClient.super.onResume();
+            BrowserFragment.super.onResume();
         }
 
         @Override
         public void superOnPause() {
-            RemoteFragmentClient.super.onPause();
+            BrowserFragment.super.onPause();
         }
 
         @Override
         public void superOnStop() {
-            RemoteFragmentClient.super.onStop();
+            BrowserFragment.super.onStop();
         }
 
         @Override
         public void superOnDestroyView() {
-            RemoteFragmentClient.super.onDestroyView();
+            BrowserFragment.super.onDestroyView();
         }
 
         @Override
         public void superOnDetach() {
-            RemoteFragmentClient.super.onDetach();
+            BrowserFragment.super.onDetach();
         }
 
         @Override
         public void superOnDestroy() {
-            RemoteFragmentClient.super.onDestroy();
+            BrowserFragment.super.onDestroy();
         }
 
         @Override
         public void superOnSaveInstanceState(IObjectWrapper outState) {
-            RemoteFragmentClient.super.onSaveInstanceState(ObjectWrapper.unwrap(outState,
-                    Bundle.class));
+            BrowserFragment.super.onSaveInstanceState(ObjectWrapper.unwrap(outState, Bundle.class));
         }
 
         @Override
         public IObjectWrapper getActivity() {
-            return ObjectWrapper.wrap(RemoteFragmentClient.this.getActivity());
+            return ObjectWrapper.wrap(BrowserFragment.this.getActivity());
         }
     };
 
@@ -102,8 +96,8 @@ public final class RemoteFragmentClient extends Fragment {
     }
 
     @Override
-    public View onCreateView(LayoutInflater inflater, ViewGroup container,
-            Bundle savedInstanceState) {
+    public View onCreateView(
+            LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
         try {
             return ObjectWrapper.unwrap(mRemoteFragment.handleOnCreateView(), View.class);
         } catch (RemoteException e) {

weblayer/public/java/org/chromium/weblayer/BrowserFragmentController.java

@@ -18,10 +18,10 @@ import org.chromium.weblayer_private.aidl.ObjectWrapper;
  */
 public final class BrowserFragmentController {
     private final IBrowserFragmentController mImpl;
-    private final RemoteFragmentClient mFragment;
+    private final BrowserFragment mFragment;
     private BrowserController mController;
 
-    BrowserFragmentController(IBrowserFragmentController impl, RemoteFragmentClient fragment) {
+    BrowserFragmentController(IBrowserFragmentController impl, BrowserFragment fragment) {
         mImpl = impl;
         mFragment = fragment;
     }

weblayer/public/java/org/chromium/weblayer/Profile.java

@@ -46,12 +46,12 @@ public final class Profile {
 
     public BrowserFragmentController createBrowserFragmentController(Context context) {
         try {
-            RemoteFragmentClient fragmentClient = new RemoteFragmentClient();
-            IBrowserFragmentController browserFragmentImpl = mImpl.createBrowserFragmentController(
-                    fragmentClient.asIRemoteFragmentClient(),
-                    ObjectWrapper.wrap(WebLayer.createRemoteContext(context)));
-            fragmentClient.setRemoteFragment(browserFragmentImpl.getRemoteFragment());
-            return new BrowserFragmentController(browserFragmentImpl, fragmentClient);
+            BrowserFragment fragment = new BrowserFragment();
+            IBrowserFragmentController browserFragmentImpl =
+                    mImpl.createBrowserFragmentController(fragment.asIRemoteFragmentClient(),
+                            ObjectWrapper.wrap(WebLayer.createRemoteContext(context)));
+            fragment.setRemoteFragment(browserFragmentImpl.getRemoteFragment());
+            return new BrowserFragmentController(browserFragmentImpl, fragment);
         } catch (RemoteException e) {
             throw new APICallException(e);
         }