• tyoshino@chromium.org's avatar
    Refactor ThreadableLoaderOptions for readability · 273dca96
    tyoshino@chromium.org authored
    ThreadableLoaderOptions shouldn't derive from ResourceLoaderOptions.
    ResourceLoaderOptions members are basically just passed through to
    FetchRequest while items added by ThreadableLoaderOptions definition
    are configuring how CORS, etc. are handled in ThreadableLoader.
    Inheritance looks making things less readable to me.
    
    Also, I'd like to make DocumentThreadbleLoader more readable by making
    constant variables held by const members.
    
    Items in ThreadableLoaderOptions that determines ThreadableLoader's
    behavior could be held by DocumentThreadableLoader as a const member.
    Items defined in ResourceLoaderOptions can be altered inside
    DocumentThreadableLoader, but can still be held as a const member by
    having members to hold overridden items.
    
    BUG=377541
    
    Review URL: https://codereview.chromium.org/301243015
    
    git-svn-id: svn://svn.chromium.org/blink/trunk@175283 bbb929c8-8fbe-4397-9dbb-9b2b20218538
    273dca96
AssociatedURLLoader.cpp 13.9 KB