• Scott Violet's avatar
    sessions: rename some sessions classes in preparation for refactor · cae997b5
    Scott Violet authored
    As a last file doesn't make sense for WebLayer this is structuring
    the code in such a way that the last file related code is not in
    the base class.
    
    This does the following:
    renames BaseSessionService* to CommandStorageManager*
    
    Next step is to move the functions related to a 'last file' out of
    CommandStorageManagerinto CommandStorageManagerWithBackup. Specifically
    the functions:
    MoveCurrentSessionToLastSession, DeleteLastSession,
    ScheduleGetLastSessionCommands. CommandStorageManagerWithBackup
    will subclass CommandStorageManager.
    
    SessionBackend will be similarly split such that functions related to last
    session are moved into SessionBackendWithBackup, which will extend
    SessionBackend.
    
    I'm not dead set on the 'WithBackup' suffix. It's not really
    a backup, so much as the last session. I considered the prefix
    'MultiFile' but that's a bit too generic and obscure. I'm certainly
    open to other suggestions.
    
    BUG=112512
    TEST=none
    
    Change-Id: I33d2c9be43bbaea4336fa32bdb3fe750d39f543a
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2005809Reviewed-by: default avatarEvan Stade <estade@chromium.org>
    Commit-Queue: Scott Violet <sky@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#732878}
    cae997b5
tab_state.cc 23.8 KB