New issue
Advanced search Search tips

Issue 689635 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Component Updater OnDemand interface must take a mandatory callback parameter

Project Member Reported by sorin@chromium.org, Feb 7 2017

Issue description

This is a code clean up issue.

At the moment, the callback is an optional parameter. This introduces branching in several call sites down the execution path and it makes it harder to reason about the code correctness.

As a design principle, the component updater code should use mandatory callbacks for the non-blocking public APIs and assert on the parameter preconditions.
 

Sign in to add a comment