New issue
Advanced search Search tips

Issue 735695 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Component updater task scheduler migration.

Project Member Reported by sorin@chromium.org, Jun 21 2017

Issue description

This is part of the refactoring of PostTask to use the new task scheduler API.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 22 2017

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

commit 31a6a2d6872b4eec83c2331a39d6cbd4f8b7b4cb
Author: Sorin Jianu <sorin@chromium.org>
Date: Thu Jun 22 16:48:58 2017

Refactoring of the component updater to use the new task scheduler api.

This is a mechanical change.

At high level:
* changing GetBlockingPool() call sites.
* Not posting to the FILE thread.

Bug:  735695 
Change-Id: If819630a8940b58c3a1e77c392297797afac9780
Reviewed-on: https://chromium-review.googlesource.com/544158
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Reviewed-by: Adam Langley <agl@chromium.org>
Commit-Queue: Sorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481564}
[modify] https://crrev.com/31a6a2d6872b4eec83c2331a39d6cbd4f8b7b4cb/chrome/browser/component_updater/pepper_flash_component_installer.cc
[modify] https://crrev.com/31a6a2d6872b4eec83c2331a39d6cbd4f8b7b4cb/chrome/browser/component_updater/pnacl_component_installer.cc
[modify] https://crrev.com/31a6a2d6872b4eec83c2331a39d6cbd4f8b7b4cb/chrome/browser/component_updater/sth_set_component_installer.cc
[modify] https://crrev.com/31a6a2d6872b4eec83c2331a39d6cbd4f8b7b4cb/chrome/browser/net/crl_set_fetcher.cc

Project Member

Comment 2 by bugdroid1@chromium.org, Jun 23 2017

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

commit c1a727127fd3126738d71722dfb39b482a9b1187
Author: Sorin Jianu <sorin@chromium.org>
Date: Fri Jun 23 16:41:28 2017

Refactoring of the iOS component updater to use the new task scheduler api.

This is a mechanical change.

At high level:
* Not posting to the FILE thread.
* Posting to a task runner obtained by calling GetTaskRunnerForThread.

Bug:  735695 
Change-Id: I273289deeb1af44868970d561717fa03615a932c
Reviewed-on: https://chromium-review.googlesource.com/544444
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Sorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481925}
[modify] https://crrev.com/c1a727127fd3126738d71722dfb39b482a9b1187/ios/chrome/browser/net/crl_set_fetcher.cc

Comment 3 by sorin@chromium.org, Jun 23 2017

Status: Fixed (was: Started)

Sign in to add a comment