New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 824770 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 708584



Sign in to add a comment

ScopedTaskEnvironment's main thread doesn't provide SequenceLocalStorageMap while in MOCK_TIME mode.

Project Member Reported by gab@chromium.org, Mar 22 2018

Issue description

It should, that's an oversight on my part. Thanks olka@ for catching this.
 

Comment 1 by gab@chromium.org, Mar 22 2018

Summary: ScopedTaskEnvironment's main thread doesn't provide SequenceLocalStorageMap while in MOCK_TIME mode. (was: ScopedTaskEnvironment doesn't provide SequenceLocalStorageMap while in MOCK_TIME mode.)
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 22 2018

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

commit 723d8fc2ac918f95458fa80802e2f9db1a3869f6
Author: Gabriel Charette <gab@chromium.org>
Date: Thu Mar 22 16:42:43 2018

Support SequenceLocalStorage from main thread of ScopedTaskEnvironment::MOCK_TIME

R=fdoray@chromium.org

Bug:  824770 
Change-Id: Ib2e8cb5ae6c3aece489d7e27c256b18c52e89cb1
Reviewed-on: https://chromium-review.googlesource.com/975781
Commit-Queue: Gabriel Charette <gab@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545111}
[modify] https://crrev.com/723d8fc2ac918f95458fa80802e2f9db1a3869f6/base/test/scoped_task_environment.cc
[modify] https://crrev.com/723d8fc2ac918f95458fa80802e2f9db1a3869f6/base/test/scoped_task_environment.h
[modify] https://crrev.com/723d8fc2ac918f95458fa80802e2f9db1a3869f6/base/test/scoped_task_environment_unittest.cc

Comment 3 by gab@chromium.org, Mar 22 2018

Status: Fixed (was: Started)

Sign in to add a comment