commit af031b69a280e10b204bbcbc111278db6ddf0186
author Bill Lin <lbill@google.com> Wed Jul 20 15:02:54 2022 +0800
committer Bill Lin <lbill@google.com> Wed Jul 20 15:17:39 2022 +0800
tree b2035509545e95c5f31c73be593ff488f1570d80
parent 60135eb28b2daefc5a9bd2f2c9f65e5def2a42f3

Only count EVENT_ONE_HANDED_TRIGGER_ROTATION_OUT in STATE_ACTIVE

The design philosophy of EVENT_ONE_HANDED_TRIGGER_ROTATION_OUT is
- When device in STATE_ACTIVE
- Rotate will automatically exit OHM

We only count when CUJ:
User trigger OHM -> Rotate -> Exit OHM

But bypass the CUJ:
No trigger OHM -> Rotate -> no action

Test: atest SystemUITests
Bug: 239654147
Change-Id: I2e73f29f4edd77998e015db960430f2b3a768349

libs/WindowManager/Shell/src/com/android/wm/shell/onehanded/OneHandedController.java

diff --git a/libs/WindowManager/Shell/src/com/android/wm/shell/onehanded/OneHandedController.java b/libs/WindowManager/Shell/src/com/android/wm/shell/onehanded/OneHandedController.java
index 1d8ac2b..42de32c 100644
--- a/libs/WindowManager/Shell/src/com/android/wm/shell/onehanded/OneHandedController.java
+++ b/libs/WindowManager/Shell/src/com/android/wm/shell/onehanded/OneHandedController.java
@@ -675,9 +675,12 @@
             return;
         }
 
+        if (mState.getState() == STATE_ACTIVE) {
+            mOneHandedUiEventLogger.writeEvent(
+                    OneHandedUiEventLogger.EVENT_ONE_HANDED_TRIGGER_ROTATION_OUT);
+        }
+
         mDisplayAreaOrganizer.onRotateDisplay(mContext, toRotation, wct);
-        mOneHandedUiEventLogger.writeEvent(
-                OneHandedUiEventLogger.EVENT_ONE_HANDED_TRIGGER_ROTATION_OUT);
     }
 
     /**