New issue
Advanced search Search tips

Issue 665224 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Some RequestCoordinatorTests failing on Lollipop Lowend tester

Project Member Reported by donnd@chromium.org, Nov 15 2016

Issue description

dougarnett@ can you take a look?

Seeing failures on this test bot for these tests.

RequestCoordinatorTest.PauseAndResumeObserver ( Code Search )
RequestCoordinatorTest.SavePageDoesStartProcessingWhenPoorlyConnected ( Code Search )

Maybe this CL has an issue on these tests on this bot?  https://codereview.chromium.org/2493683002

https://uberchromegw.corp.google.com/i/chromium.android/builders/Lollipop%20Low-end%20Tester/builds/4307

C  270.229s Main  [FAIL] RequestCoordinatorTest.PauseAndResumeObserver:
C  270.229s Main  [ RUN      ] RequestCoordinatorTest.PauseAndResumeObserver
C  270.229s Main  ../../components/offline_pages/background/request_coordinator_unittest.cc:1305: Failure
C  270.229s Main  Value of: observer().state()
C  270.229s Main    Actual: 4-byte object <00-00 00-00>
C  270.229s Main  Expected: SavePageRequest::RequestState::OFFLINING
C  270.229s Main  Which is: 4-byte object <02-00 00-00>
C  270.229s Main  [  FAILED  ] RequestCoordinatorTest.PauseAndResumeObserver (4 ms)
C  270.229s Main  [FAIL] RequestCoordinatorTest.SavePageDoesStartProcessingWhenPoorlyConnected:
C  270.229s Main  [ RUN      ] RequestCoordinatorTest.SavePageDoesStartProcessingWhenPoorlyConnected
C  270.229s Main  ../../components/offline_pages/background/request_coordinator_unittest.cc:1413: Failure
C  270.230s Main  Value of: is_busy()
C  270.230s Main    Actual: false
C  270.230s Main  Expected: true
C  270.230s Main  [  FAILED  ] RequestCoordinatorTest.SavePageDoesStartProcessingWhenPoorlyConnected (1 ms)

 
Status: Assigned (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ed80d65c5b1d36d306c02aadf7e9594701df7152

commit ed80d65c5b1d36d306c02aadf7e9594701df7152
Author: dougarnett <dougarnett@chromium.org>
Date: Wed Nov 16 00:06:20 2016

[OfflinePages] Fix 2 RequestCoordinator tests for low end devices

BUG= 665224 

Review-Url: https://codereview.chromium.org/2503073002
Cr-Commit-Position: refs/heads/master@{#432299}

[modify] https://crrev.com/ed80d65c5b1d36d306c02aadf7e9594701df7152/components/offline_pages/background/request_coordinator_unittest.cc

Status: Fixed (was: Assigned)

Sign in to add a comment