• A Olsen's avatar
    Remove BrowserProcess dep in c/b/chromeos/settings · b16a5e53
    A Olsen authored
    DeviceSettingsProvider in chrome/browser/chromeos/settings has an
    undesirable dependency on g_browser_process, which it uses to access
    the LocalState singleton. This CL removes the dependency on
    g_browser_process by passing in LocalState explicitly.
    
    Note that the tests and test helpers still depend on BrowserProcess
    or TestingBrowserProcess, which they use to initialize and access the
    LocalState. For tests in the settings package, I need to find a way to
    remove this dependency in a follow up CL, so that the settings package
    along with tests & test helpers can be moved out of chrome/browser.
    
    See go/cros-untangle2
    
    Bug: 446937
    Change-Id: If9b14940f05ceef1fa8f33e612df22d3d9173d8f
    Reviewed-on: https://chromium-review.googlesource.com/1233711Reviewed-by: default avatarJochen Eisinger <jochen@chromium.org>
    Reviewed-by: default avatarMaksim Ivanov <emaxx@chromium.org>
    Reviewed-by: default avatarSteven Bennetts <stevenjb@chromium.org>
    Commit-Queue: A Olsen <olsen@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#593199}
    b16a5e53
device_settings_provider.h 5.77 KB