New issue
Advanced search Search tips

Issue 598476 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Sep 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug

Blocking:
issue 589526



Sign in to add a comment

Offline Pages: Unit test asynchronous API

Project Member Reported by dewittj@chromium.org, Mar 28 2016

Issue description

Async apis are being added to OfflinePageBridge; these should be unit tested as well as tested with Instrumentation Tests.
 
Blocking: 589526
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 29 2016

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

commit 2f961ea9dae09bab5060fa8bb80584144f11276b
Author: fgorski <fgorski@chromium.org>
Date: Tue Mar 29 22:09:30 2016

[OfflinePages] Adding tests to OfflinePageBridge#getAllPagesAsync

Adds a few junit tests for the getAllPagesAsync as a template for other
asynchronous functions on OfflinePageBridge.

BUG= 598476 ,595252
R=dewittj@chromium.org

Review URL: https://codereview.chromium.org/1841103002

Cr-Commit-Position: refs/heads/master@{#383844}

[modify] https://crrev.com/2f961ea9dae09bab5060fa8bb80584144f11276b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java
[modify] https://crrev.com/2f961ea9dae09bab5060fa8bb80584144f11276b/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridgeTest.java

Comment 3 by dim...@chromium.org, Sep 27 2016

Status: WontFix (was: Assigned)

Sign in to add a comment