Actually force a reload when the CommChannel tells the page to reload.

Review Request #13273 — Created Sept. 12, 2023 and submitted

Information

Review Board
release-6.x

Reviewers

The new communication channel implementation allows one tab to tell
others to reload themselves, but I had implemented that by basically
doing window.location = window.location. This was fine for most cases,
but if there was an active anchor in the URL, the page would just scroll
to that anchor. This change fixes it to actually call
window.location.reload().

  • Verified that reloading worked correctly.
  • Ran unit tests.
Summary ID
Actually force a reload when the CommChannel tells the page to reload.
The new communication channel implementation allows one tab to tell others to reload themselves, but I had implemented that by basically doing `window.location = window.location`. This was fine for most cases, but if there was an active anchor in the URL, the page would just scroll to that anchor. This change fixes it to actually call `window.location.reload()`. Testing Done: - Verified that reloading worked correctly. - Ran unit tests.
a420bf74a406ea511e46b5987a7b82881f10fadb
maubin
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to release-6.x (2365d95)