• Wei Li's avatar
    [MacViews] Track fullscreen bookmark bar state correctly · 9a7bf866
    Wei Li authored
    Mac browser fullscreen mode allows to keep the top UI, as well as
    toggling between showing and hiding the top UI. In these cases, bookmark
    bar should always show or hide in sync with the top UI. This requires
    tracking top UI change, and change bookmark bar state or visibility
    accordingly.
    
    In Views, bookmark bar state is cached in Browser::bookmark_bar_state_,
    and is updated through Browser::UpdateBookmarkBarState(). So when the
    visibility of top UI changes, we need to execute
    UpdateBookmarkBarState() to update bookmark bar state as well.
    
    BUG=831219
    TEST=pls check bookmark bar (non-detached) in fullscreen mode, which
         should always show or hide along with toolbar/tabstrip.
    
    Change-Id: I04475edc455a62e045857ed666c44a31ead2b402
    Reviewed-on: https://chromium-review.googlesource.com/1109405
    Commit-Queue: Wei Li <weili@chromium.org>
    Reviewed-by: default avatarTrent Apted <tapted@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#570324}
    9a7bf866
browser.cc 103 KB