Hi Reilly, are you looking at just external_data_policy_fetcher.cc as part of this? There is a bunch of cloudprint stuff that sort of awkwardly got bugfolded into this which may need to have its bug resurrected if that's the case..
Status: Started (was: Assigned) Summary: Migrate components/policy/core/common/cloud to network::SimpleURLLoader (was: Migrate components/policy/core/common/cloud/external_policy_data_fetcher.cc)
I originally merged issue 844952 into this issue because I assumed that updating one class in components/policy/core/common/cloud to network::SimpleURLLoader would probably lead to migrating another. I'm not entirely sure yet but I think UserInfoFetcher and ExternalPolicyDataFetcher are actually reasonably independent. That change for the latter is getting a little large. I will probably work on UserInfoFetcher separately.
I am not aware of the manual testing steps for this feature. All existing unit tests pass. This work was not fixing a bug, but migrating existing code to a new API.
Comment 1 by dxie@google.com
, May 20 2018Status: Available (was: Untriaged)