New issue
Advanced search Search tips

Issue 702326 link

Starred by 1 user

Issue metadata

Status: Duplicate
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug

Blocking:
issue 670879



Sign in to add a comment

unit_test.GeolocationPermissionContextTests test fails when run alone on android

Project Member Reported by bpastene@chromium.org, Mar 16 2017

Issue description

Specifically the test GeolocationPermissionContextTests.GeolocationStatusSystemDisabled fails when run alone.

That is, this passes:
python build/android/test_runner.py gtest --suite unit_tests --output-directory out/Release -v

And this fails (same chrm rev):
python build/android/test_runner.py gtest --suite unit_tests --output-directory out/Release -v -f GeolocationPermissionContextTests.GeolocationStatusSystemDisabled

The test complains with:
I   39.554s Main  1 failed tests remain.
C   39.555s Main  ********************************************************************************
C   39.555s Main  Detailed Logs
C   39.555s Main  ********************************************************************************
C   39.555s Main  [FAIL] GeolocationPermissionContextTests.GeolocationStatusSystemDisabled:
C   39.555s Main  [ RUN      ] GeolocationPermissionContextTests.GeolocationStatusSystemDisabled
C   39.555s Main  ../../chrome/browser/geolocation/geolocation_permission_context_unittest.cc:1140: Failure
C   39.555s Main  Value of: PermissionManager::Get(profile())->GetPermissionStatus( content::PermissionType::GEOLOCATION, requesting_frame, requesting_frame)
C   39.556s Main    Actual: PermissionStatus::DENIED
C   39.556s Main  Expected: blink::mojom::PermissionStatus::ASK
C   39.556s Main  Which is: PermissionStatus::{ASK, LAST}
C   39.556s Main  [  FAILED  ] GeolocationPermissionContextTests.GeolocationStatusSystemDisabled (82 ms)
C   39.556s Main  [----------] 1 test from GeolocationPermissionContextTests (83 ms total)


This will block us from arbitrarily sharding all android test suites. The test was added in https://codereview.chromium.org/2743603002, so assigning to author.
 
Blocking: 670879
Mergedinto: 701719
Status: Duplicate (was: Untriaged)
Fixing now.

Sign in to add a comment