Issue 724293 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	pmonette@chromium.org
Closed:	Jun 2017
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	3
Type:	Bug

ProtocolHandlerRegistryTest crashes when all tasks get flushed

Project Member Reported by pmonette@chromium.org, May 18 2017

Issue description

When these tests are changed to flush all remaining task in
the task scheduler, it crashes on a use-after-free.

This prevent the migration of the BrowserThread::FILE.

Project Member

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

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

commit fd04fe52772ad91203606d3797d5e7a8b6226c04
Author: pmonette <pmonette@chromium.org>
Date: Fri May 19 22:04:25 2017

Remove FILE thread dependency in protocol_handler_registry_unittest.cc

BUG= 724293 ,  689520 

Review-Url: https://codereview.chromium.org/2896473002
Cr-Commit-Position: refs/heads/master@{#473349}

[modify] https://crrev.com/fd04fe52772ad91203606d3797d5e7a8b6226c04/chrome/browser/custom_handlers/protocol_handler_registry.cc
[modify] https://crrev.com/fd04fe52772ad91203606d3797d5e7a8b6226c04/chrome/browser/custom_handlers/protocol_handler_registry.h
[modify] https://crrev.com/fd04fe52772ad91203606d3797d5e7a8b6226c04/chrome/browser/custom_handlers/protocol_handler_registry_unittest.cc
[modify] https://crrev.com/fd04fe52772ad91203606d3797d5e7a8b6226c04/chrome/browser/ui/content_settings/content_setting_bubble_model_unittest.cc

Comment 2 by pmonette@chromium.org, Jun 1 2017

Status: Fixed (was: Assigned)

►

Issue 724293 link

Issue metadata

ProtocolHandlerRegistryTest crashes when all tasks get flushed

Issue description

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

Comment 1 Deleted

Comment 2 by pmonette@chromium.org, Jun 1 2017

Comment 2 Deleted

Sign in to add a comment