New issue
Advanced search Search tips

Issue 742489 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

DCHECK_CURRENTLY_ON misleading in download_history_unittest.cc

Project Member Reported by btolsch@chromium.org, Jul 13 2017

Issue description

All of the uses of DCHECK_CURRENTLY_ON will pass with any thread label (UI, IO, etc.) because TestBrowserThreadBundle uses the same thread and MessageLoop for all threads.

The code doesn't seem to depend on any thread switching though, so this is mostly an FYI.
 
Owner: dtrainor@chromium.org
Status: Assigned (was: Untriaged)

Sign in to add a comment