• Luciano Pacheco's avatar
    Fix Polymer config for Video Player · ecb6e0bb
    Luciano Pacheco authored
    In Video Player polymer_config.js was writing to console.error:
    polymer_config.js:15 Polymer is already defined.
    
    This was caused by <paper-slider.html> being included before any of
    WebUI or Files app Polymer element had a chance to initialize with the
    right config (polymer.html), which in turn was causing Polymer elements
    to render without shadowRoot.
    
    Change <files_icon_button> to include before <paper-slider> so it loads
    the right config/polymer.html.
    
    Test: --gtest_filter='VideoPlayerBrowserTest.CheckInitialElements' also manually checked for no error message and play button has shadowRoot.
    Bug: 875525
    Change-Id: I9888992cce762cd6eae8c31a8042119506fd5590
    Reviewed-on: https://chromium-review.googlesource.com/1248342
    Commit-Queue: Luciano Pacheco <lucmult@chromium.org>
    Reviewed-by: default avatarTrent Apted <tapted@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#594620}
    ecb6e0bb
video_player.html 2.21 KB