New issue
Advanced search Search tips

Issue 809255 link

Starred by 1 user
Status: Available
Owner: ----
Cc:
mek@chromium.org
dmu...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 920322



Sign in to add a comment

Remove content::StorageAreaImpl's dependency on ThreadTaskRunnerHandle

Project Member Reported by dmu...@chromium.org, Feb 5 2018 
LevelDBWrapper uses base::ThreadTaskRunnerHandle::Get() to start the commit timer.

This adds a hard dependency on using a SingleThreadTaskRunner. In order to move dom storage's mojo execution to a sequencedtaskrunner, this should be either given a task runner or something else.

Comment 1 by mek@chromium.org, Feb 5 2018 

Probably just switching to base::SequencedTaskRunnerHandle::Get() is good enough without needing to pass around explicit task runners?

Comment 2 by dmu...@chromium.org, Feb 7 2018 

Awesome! I didn't know that existed :P

Comment 3 by dmu...@chromium.org, Jan 3

Cc: dmu...@chromium.org
Labels: Hotlist-GoodFirstBug
Owner: ----
Status: Available (was: Assigned)

Comment 4 by mek@chromium.org, Jan 3

Summary: Remove content::StorageAreaImpl's dependency on ThreadTaskRunnerHandle (was: Remove LevelDBWrapper's dependency on ThreadTaskRunnerHandle)

Comment 5 by dmu...@chromium.org, Jan 9

Blocking: 920322
►

Sign in to add a comment