• Danyao Wang's avatar
    [Nav Experiment] Cache visible item in nav manager during session restore · b7cfeb48
    Danyao Wang authored
    TabGridMediator relies on VisibleURL to determine if the title of a
    tab should be hidden. During session restore, the restore_session.html
    item is briefly the only visible item in the web view until the other
    entries are restored, and it has chrome://newtab as the virtual URL.
    This incorrectly causes the CreateItem() in TabGridMediator to set
    |hidesTitle| to true for the grid item.
    
    Previously visible item title was cached in WebStateImpl for a similar
    reason (http://crbug.com/819606). This change combines the two use cases.
    
    
    Bug: 869351
    Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
    Change-Id: I5391eda466ad9de26087d2d8a1e2a97a7a83f27a
    Reviewed-on: https://chromium-review.googlesource.com/1157866
    Commit-Queue: Danyao Wang <danyao@chromium.org>
    Reviewed-by: default avatarEugene But <eugenebut@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#581946}
    b7cfeb48
wk_based_navigation_manager_impl.h 11 KB