Browser hanging (high CPU) if theme changed without refreshing page

gadgetchnnel commented 5 years ago

Describe the issue:

With Compact Custom Header enabled, the browser will hang if the following steps are performed:

Click on Profile
Change the theme from the dropdown
Click on Overview to navigate to the Lovelace home page
Click on any other tab or sidebar icon

When it hangs, the CPU usage of Chrome rapidly increases up to almost 200%. The page is unresponsive until the process is killed (although sometimes, if you catch it quickly enough, you can close the tab instead). The same happens with Safari (the CPU only seems to increase to around 100% but the experience is the same). Using the Chrome browser on an Android phone, the whole browser becomes unresponsive and needs to be force stopped.

If Compact Custom Header is disabled or the page is refreshed before step 4 it doesn't happen.

Disabling Compact Custom Header stops this from happening. It also doesn’t happen if I refresh the page after changing the theme (as long as this is done before trying to navigate to another tab).

Config YAML:

cch:
  chevrons: true
  exceptions:
    - conditions:
        user: Kiosk
      config:
        default_tab: Kiosk
        disable: false
        header: true
        hide_tabs: 0 to 8
        kiosk_mode: true
        menu: show
        notifications: show
        options: show
        redirect: true
        show_tabs: Kiosk
        voice: show
        disable_sidebar: false
    - conditions:
        query_string: embed
      config:
        default_tab: []
        hide_tabs: '9'
        kiosk_mode: true
        menu: show
        notifications: show
        options: show
        show_tabs: []
        voice: show
    - conditions:
        user_agent: Mobile
      config:
        default_tab: []
        hide_tabs: 'Kiosk,Calendar,media-players,Github,Test'
        menu: show
        notifications: show
        options: show
        show_tabs: []
        voice: overflow
  hide_help: true
  hide_tabs: 'Kiosk,Calendar,Test'
  swipe: true