Fix INJECT_EVENTS permission causing VR e2e tests to fail on K |
|||
Issue descriptionRarely, testScreenTapsRegisteredOnCardboard will fail on KitKat due to not having the INJECT_EVENTS permission (see https://build.chromium.org/p/chromium.fyi/builders/Android%20VR%20Tests/builds/6571). Why this only happens rarely instead of all the time is unclear. Using UI Automator instead of Instrumentation may fix this, as according to https://developer.android.com/reference/android/app/Instrumentation.html#getUiAutomation(), using it to inject events should work regardless of the current application.
,
Apr 12 2017
Seems to be an issue on L as well...
,
Apr 12 2017
It seems there is a race condition issue between the second screen tap we send to chrome and the confirmation dialog. Will the confirmation dialog show up always if we don't send the second screen tap to Chrome? Is it expected? If yes, how about waiting until the confirmation dialog shows up and close it, then send the second screen tap to Chrome?
,
Apr 12 2017
The confirmation dialog will only show up if it's never been confirmed before on the device. However, it looks like the device provisioning script somehow disables it, as it no longer appears during tests after provisioning. I would guess that it's this setting https://cs.chromium.org/chromium/src/third_party/catapult/devil/devil/android/settings.py?q=package:%5E(chromium)$+file:(/%7C%5E)devil/android/settings(%5C.(swig%7Cpy%7Cspt)$%7C/(__init__%5C.(swig%7Cpy%7Cspt))?$)&dr&l=83. However, since it's disabled during tests, I'm not sure why this is happening at all, as all the swarming devices should be provisioned properly. I'll check with John on this.
,
Apr 12 2017
This is starting to look like a swarming problem. I found the bot/devices that this failed on at one point and ran a script as a swarming test that dumps the content://settings/secure table from the devices: https://isolateserver.appspot.com/browse?namespace=default-gzip&hash=7c196ea8594436e9d5b4100e105176eb88359a37 The first device has anr_show_background set to 0, but the rest don't. I'll double check that the devices it does pass on have it set to make sure that's actually the issue, then file a bug to get the devices re-provisioned.
,
Apr 13 2017
Confirmed to be a swarming issue in Issue 711035 - apparently anr_show_background isn't currently set during swarming provisioning https://chrome-internal.googlesource.com/infradata/config/+/master/configs/chromium-swarm/scripts/android.py#227. bpastene@ has agreed to add it to the list of content://settings/secure settings to set during provisioning, so I'll mark this as closed after that goes in.
,
Apr 14 2017
Closing this since anr_show_background setting has been added to swarming bots' provisioning.
,
Jul 4
|
|||
►
Sign in to add a comment |
|||
Comment 1 by bsheedy@chromium.org
, Apr 12 2017