• Sunggook Chue's avatar
    Refactor of moving shortcut deletion to AppShortcutManager · 50ba991a
    Sunggook Chue authored
    OsIntegrationManager is a proxy class that doesn't have to know
    its detailed OS calls instead it delegates a work to the target
    class if it exists.
    
    AppShortcutManager handles shortcut creation (see CreateShortcuts
    in OsIntegrationManager) that includes proper telemetry message
    at the end of it. However the current DeleteShortcuts is simply
    handled by the OsItengrationManager instead of AppShortcutManager.
    
    The change is to follow pattern of CreateShortcuts for the shortcut
    deletion that seems more consistent in terms of OsItegrationManager
    and AppShortcutManager relationship.
    
    Bug: 1151796
    Change-Id: I01e80b0a13ef1169a238d65a2ffc5acb6cca292f
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2606680Reviewed-by: default avatarDaniel Murphy <dmurph@chromium.org>
    Reviewed-by: default avatarRobert Kaplow <rkaplow@chromium.org>
    Commit-Queue: Daniel Murphy <dmurph@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#841612}
    50ba991a
histograms.xml 691 KB