New issue
Advanced search Search tips

Issue 724547 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

write instrumentation test for cookie manager start up path

Project Member Reported by boliu@chromium.org, May 19 2017

Issue description

to make sure regression like  crbug.com/723836  is caught by cq
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 25 2017

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

commit f90384eb071fcc77d7aa82d34f8c5a71fc27919d
Author: Bo Liu <boliu@chromium.org>
Date: Thu May 25 17:06:18 2017

aw: Call setWillOverrideUiThread in CookieManagerStartupTest

CookieManager can be started before a UI thread is set, and this change
makes sure the CookieManagerStartupTest similuates this environment too.

Made sure that cherry-picking
https://chromium-review.googlesource.com/c/503516/ crashes a test.

BUG= 724547 

Change-Id: If69b834cff64f1ebdd59e664bfaf2401c5c0fd62
Reviewed-on: https://chromium-review.googlesource.com/513238
Reviewed-by: Richard Coles <torne@chromium.org>
Commit-Queue: Bo Liu <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#474681}
[modify] https://crrev.com/f90384eb071fcc77d7aa82d34f8c5a71fc27919d/android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerStartupTest.java

Comment 2 by boliu@chromium.org, May 25 2017

Owner: boliu@chromium.org
Status: Fixed (was: Available)

Sign in to add a comment