Adds force load for load url

Resolves brave#13584 Auditors: Test Plan:
NejcZdovc · Mar 27, 2018 · 0620003 · 0620003
1 parent f482935
commit 0620003
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 6 deletions.
diff --git a/app/browser/tabs.js b/app/browser/tabs.js
@@ -860,19 +860,25 @@ const api = {
     const tabId = action.get('tabId')
     const tab = webContentsCache.getWebContents(tabId)
     if (tab && !tab.isDestroyed()) {
-      let url = normalizeUrl(action.get('url'))
+      const url = normalizeUrl(action.get('url'))
+      const currentUrl = tab.getURL()
       // We only allow loading URLs explicitly when the origin is
       // the same for pinned tabs.  This is to help preserve a users
       // pins.
-      if (tab.pinned && getOrigin(tab.getURL()) !== getOrigin(url)) {
+      if (tab.pinned && getOrigin(currentUrl) !== getOrigin(url)) {
         api.create({
           url,
           partition: tab.session.partition
         })
         return
       }
 
-      tab.loadURL(url)
+      const reloadMatchingUrl = action.get('reloadMatchingUrl') || false
+      if (reloadMatchingUrl && currentUrl === url) {
+        tab.reload(true)
+      } else {
+        tab.loadURL(url)
+      }
     }
   },
 

diff --git a/app/renderer/components/preferences/payment/enabledContent.js b/app/renderer/components/preferences/payment/enabledContent.js
@@ -226,7 +226,8 @@ class EnabledContent extends ImmutableComponent {
   recoverStatusClick () {
     appActions.loadURLRequested(
       parseInt(this.props.ledgerData.get('tabId')),
-      'about:preferences#payments?ledgerRecoveryOverlayVisible'
+      'about:preferences#payments?ledgerRecoveryOverlayVisible',
+      true
     )
   }
 

diff --git a/js/actions/appActions.js b/js/actions/appActions.js
@@ -255,12 +255,14 @@ const appActions = {
    * A request for a URL load
    * @param {number} tabId - the tab ID to load the URL inside of
    * @param {string} url - The url to load
+   * @param {boolean} reloadMatchingUrl - would you like to force reload provided tab
    */
-  loadURLRequested: function (tabId, url) {
+  loadURLRequested: function (tabId, url, reloadMatchingUrl) {
     dispatch({
       actionType: appConstants.APP_LOAD_URL_REQUESTED,
       tabId,
-      url
+      url,
+      reloadMatchingUrl
     })
   },