Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8be17739 authored by Joanne Chung's avatar Joanne Chung
Browse files

[DO NOT MERGE] Revert "Revert "[DO NOT MERGE] Remove selection toolbar code from build time"" 

This reverts commit 888e79f7.

Reason for revert: The feature is planned in QPR2. This should be removed before QPR1 deadline.

Change-Id: I8af811935171d7ff51d9ba2a9ea60437695d7966
parent 888e79f7

core/java/Android.bp

+8 −0
Original line number Diff line number Diff line
@@ -15,6 +15,14 @@ filegroup { 
        "**/*.java",

        "**/*.aidl",

    ],

    exclude_srcs: [

        // Remove election toolbar code from build time

        "android/service/selectiontoolbar/*.aidl",

        "android/service/selectiontoolbar/*.java",

        "android/view/selectiontoolbar/*.aidl",

        "android/view/selectiontoolbar/*.java",

        "com/android/internal/widget/floatingtoolbar/RemoteFloatingToolbarPopup.java",

    ],

    visibility: ["//frameworks/base"],

}

core/java/android/app/SystemServiceRegistry.java

+0 −11
Original line number Diff line number Diff line
@@ -230,8 +230,6 @@ import android.view.contentcapture.ContentCaptureManager; 
import android.view.contentcapture.IContentCaptureManager;

import android.view.displayhash.DisplayHashManager;

import android.view.inputmethod.InputMethodManager;

import android.view.selectiontoolbar.ISelectionToolbarManager;

import android.view.selectiontoolbar.SelectionToolbarManager;

import android.view.textclassifier.TextClassificationManager;

import android.view.textservice.TextServicesManager;

import android.view.translation.ITranslationManager;
@@ -365,15 +363,6 @@ public final class SystemServiceRegistry { 
                return new TextClassificationManager(ctx);

            }});



        registerService(Context.SELECTION_TOOLBAR_SERVICE, SelectionToolbarManager.class,

                new CachedServiceFetcher<SelectionToolbarManager>() {

                    @Override

                    public SelectionToolbarManager createService(ContextImpl ctx) {

                        IBinder b = ServiceManager.getService(Context.SELECTION_TOOLBAR_SERVICE);

                        return new SelectionToolbarManager(ctx.getOuterContext(),

                                ISelectionToolbarManager.Stub.asInterface(b));

                    }});



        registerService(Context.FONT_SERVICE, FontManager.class,

                new CachedServiceFetcher<FontManager>() {

            @Override

core/java/com/android/internal/widget/floatingtoolbar/FloatingToolbarPopup.java

+1 −5
Original line number Diff line number Diff line
@@ -21,7 +21,6 @@ import android.content.Context; 
import android.graphics.Rect;

import android.view.MenuItem;

import android.view.View;

import android.view.selectiontoolbar.SelectionToolbarManager;

import android.widget.PopupWindow;



import java.util.List;
@@ -93,10 +92,7 @@ public interface FloatingToolbarPopup { 
     * enabled, otherwise returns {@link LocalFloatingToolbarPopup} implementation.

     */

    static FloatingToolbarPopup createInstance(Context context, View parent) {

        boolean enabled = SelectionToolbarManager.isRemoteSelectionToolbarEnabled(context);

        return enabled

                ? new RemoteFloatingToolbarPopup(context, parent)

                : new LocalFloatingToolbarPopup(context, parent);

        return new LocalFloatingToolbarPopup(context, parent);

    }



}

services/Android.bp

+0 −2
Original line number Diff line number Diff line
@@ -102,7 +102,6 @@ filegroup { 
        ":services.profcollect-sources",

        ":services.restrictions-sources",

        ":services.searchui-sources",

        ":services.selectiontoolbar-sources",

        ":services.smartspace-sources",

        ":services.speech-sources",

        ":services.systemcaptions-sources",
@@ -158,7 +157,6 @@ java_library { 
        "services.profcollect",

        "services.restrictions",

        "services.searchui",

        "services.selectiontoolbar",

        "services.smartspace",

        "services.speech",

        "services.systemcaptions",

services/java/com/android/server/SystemServer.java

+0 −7
Original line number Diff line number Diff line
@@ -338,8 +338,6 @@ public final class SystemServer implements Dumpable { 
            "com.android.server.contentcapture.ContentCaptureManagerService";

    private static final String TRANSLATION_MANAGER_SERVICE_CLASS =

            "com.android.server.translation.TranslationManagerService";

    private static final String SELECTION_TOOLBAR_MANAGER_SERVICE_CLASS =

            "com.android.server.selectiontoolbar.SelectionToolbarManagerService";

    private static final String MUSIC_RECOGNITION_MANAGER_SERVICE_CLASS =

            "com.android.server.musicrecognition.MusicRecognitionManagerService";

    private static final String SYSTEM_CAPTIONS_MANAGER_SERVICE_CLASS =
@@ -2636,11 +2634,6 @@ public final class SystemServer implements Dumpable { 
            Slog.d(TAG, "TranslationService not defined by OEM");

        }



        // Selection toolbar service

        t.traceBegin("StartSelectionToolbarManagerService");

        mSystemServiceManager.startService(SELECTION_TOOLBAR_MANAGER_SERVICE_CLASS);

        t.traceEnd();



        // NOTE: ClipboardService depends on ContentCapture and Autofill

        t.traceBegin("StartClipboardService");

        mSystemServiceManager.startService(ClipboardService.class);