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

Commit 0d7dc3b8 authored by Joe Onorato's avatar Joe Onorato
Browse files

Fix notification bar performance.

Bug 2195678

We had all this stuff there to keep the notification view
around so it doesn't need to be reconstructed each time,
and we weren't using it.  This fixes that, and it looks
much better now.
parent 9efd523c
Loading
Loading
Loading
Loading
+9 −5
Original line number Diff line number Diff line
@@ -950,7 +950,8 @@ public class StatusBarService extends IStatusBar.Stub
        panelSlightlyVisible(true);
        
        updateExpandedViewPos(EXPANDED_LEAVE_ALONE);
        mExpandedDialog.show();
        mExpandedParams.flags &= ~WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE;
        mExpandedDialog.getWindow().setAttributes(mExpandedParams);
        mExpandedView.requestFocus(View.FOCUS_FORWARD);
        mTrackingView.setVisibility(View.VISIBLE);
        
@@ -1027,7 +1028,8 @@ public class StatusBarService extends IStatusBar.Stub
        }
        mExpandedVisible = false;
        panelSlightlyVisible(false);
        mExpandedDialog.hide();
        mExpandedParams.flags |= WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE;
        mExpandedDialog.getWindow().setAttributes(mExpandedParams);
        mTrackingView.setVisibility(View.GONE);

        if ((mDisabled & StatusBarManager.DISABLE_NOTIFICATION_ICONS) == 0) {
@@ -1056,6 +1058,7 @@ public class StatusBarService extends IStatusBar.Stub
            else if (mAnimY < mStatusBarView.getHeight()) {
                if (SPEW) Log.d(TAG, "Animation completed to collapsed state.");
                mAnimating = false;
                updateExpandedViewPos(0);
                performCollapse();
            }
            else {
@@ -1508,17 +1511,19 @@ public class StatusBarService extends IStatusBar.Stub
            }
        }

        final int disph = mDisplay.getHeight();
        lp = mExpandedDialog.getWindow().getAttributes();
        lp.width = ViewGroup.LayoutParams.FILL_PARENT;
        lp.height = ViewGroup.LayoutParams.WRAP_CONTENT;
        lp.x = 0;
        lp.y = 0;
        mTrackingPosition = lp.y = -disph; // sufficiently large negative
        lp.type = WindowManager.LayoutParams.TYPE_STATUS_BAR_PANEL;
        lp.flags = WindowManager.LayoutParams.FLAG_LAYOUT_IN_SCREEN
                | WindowManager.LayoutParams.FLAG_LAYOUT_NO_LIMITS
                | WindowManager.LayoutParams.FLAG_NOT_TOUCH_MODAL
                | WindowManager.LayoutParams.FLAG_ALT_FOCUSABLE_IM
                | WindowManager.LayoutParams.FLAG_DITHER;
                | WindowManager.LayoutParams.FLAG_DITHER
                | WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE;
        lp.format = pixelFormat;
        lp.gravity = Gravity.TOP | Gravity.FILL_HORIZONTAL;
        lp.setTitle("StatusBarExpanded");
@@ -1531,7 +1536,6 @@ public class StatusBarService extends IStatusBar.Stub
                new ViewGroup.LayoutParams(ViewGroup.LayoutParams.FILL_PARENT,
                                           ViewGroup.LayoutParams.WRAP_CONTENT));
        mExpandedDialog.show();
        mExpandedDialog.hide();
        FrameLayout hack = (FrameLayout)mExpandedView.getParent();
        hack.setForeground(null);
    }
+0 −2
Original line number Diff line number Diff line
@@ -55,8 +55,6 @@ public class TrackingPatternView extends View {
        final int textureWidth = mTextureWidth;
        final int textureHeight = mTextureHeight;

        Log.d("TrackingPatternView", "width=" + width + " textureWidth=" + textureWidth);

        int x = 0;
        int y;