New issue
Advanced search Search tips

Issue 923444 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 644355



Sign in to add a comment

ash::network_icon: Elim NetworkHandler dependencies

Project Member Reported by steve...@chromium.org, Jan 18 (4 days ago)

Issue description

Currently ash::network_icon makes liberal use of calls to the NetworkHandler singleton instance. This is scheduled to be replaced with a mojo API in Mash.

Since NetworkIcon is a series of utility functions and not a class, the code needs to be re-factored to either be a class with a connection to the mojo API, or the properties it needs should be entirely encapsulated into the NetworkState instances passed to it, which would allow a straightforward replacement of chromeos::NetworkState with chromeos::network_config::mojom::NetworkStateProperties.

 

Comment 1 by steve...@chromium.org, Jan 18 (4 days ago)

Blocking: 644355

Sign in to add a comment