• Mark Cogan's avatar
    [iOS] Migrate away from downstream URLLoadingService use (6/10) · e3bb56ab
    Mark Cogan authored
    As part of preparing for multi-window, the URLLoadingService needs
    to be migrated to be a BrowserAgent instead of a KeyedService. That
    change is itself fairly large, but as an additional complication
    URLLoadingService is used in downstream code.
    
    This CL is the sixth in a series of changes needed to complete this
    refactor:
    
    1. (crrev.com/c/2089687) Add a new provider API upstream that passes
       both a Browser and temporary URL loading shim to CreateLogoVendor.
    2. Implement this new API downstream and add support for initializing
       the LogoController with both Browser and the loading shim. Use the
       loading shim for loading URLs if it is present.
    3. (crrev.com/c/2089785) Upstream, call the new CreateLogoVendor() API.
    4. Downstream, remove support for the old CreateLogoVendor() API and
       update LogoController to only be initialized with a Browser and the
       shim loader. Remove downstream use of the URLLoadingService.
    5. (crrev.com/c/2105314) Upstream, refactor URLLoadingService to be
       URLLoadingBrowserAgent, updating the upstream implementation of the
       loading shim passed downstream in CreateLogoVendor().
    6. Upstream, add a new provider API that no longer passes the loading
       shim into CreateLogoVendor(). Delete the now-unused provider API that
       was removed downstream in step 4.
    7. Downstream, remove use of the loading shim from LogoController and
       have it fetch the URLLoadingBrowserAgent from its Browser parameter.
       Implement the new shim-less CreateLogoVendor() API. Have the old
       shimmed API call the new shimless one.
    8. Upstream, call only the new shimless API. Remove the shim
       implementation in calling code.
    9. Downstream, remove the shimmed API.
    10.Upstream, remove the shimmed API, including the shim definition.
    
    Bug: 1046374
    Change-Id: I12874b8c0d6674b03cee22141b67bc32788fa4a5
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106182
    Commit-Queue: Mark Cogan <marq@chromium.org>
    Reviewed-by: default avatarStepan Khapugin <stkhapugin@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#751316}
    e3bb56ab
chrome_browser_provider.h 6.67 KB