Bug 662155 - addTab should report the exception when loadURIWithFlags fails. r=gavin
authorDão Gottwald <dao@mozilla.com>
Sun, 05 Jun 2011 19:37:10 +0200
changeset 70570 93911949517ca791ecc1749f199099154bc0a987
parent 70569 e9a667cf1687a66c220906817e7a8dca249870a1
child 70571 64dd0ffff3f20f2ec4a8c7b0c0734d9cb274d046
push id20360
push userdgottwald@mozilla.com
push dateSun, 05 Jun 2011 17:37:54 +0000
treeherdermozilla-central@93911949517c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgavin
bugs662155
milestone7.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 662155 - addTab should report the exception when loadURIWithFlags fails. r=gavin
browser/base/content/tabbrowser.xml
--- a/browser/base/content/tabbrowser.xml
+++ b/browser/base/content/tabbrowser.xml
@@ -1329,19 +1329,19 @@
 
               let flags = Ci.nsIWebNavigation.LOAD_FLAGS_NONE;
               if (aAllowThirdPartyFixup)
                 flags |= Ci.nsIWebNavigation.LOAD_FLAGS_ALLOW_THIRD_PARTY_FIXUP;
               if (aFromExternal)
                 flags |= Ci.nsIWebNavigation.LOAD_FLAGS_FROM_EXTERNAL;
               try {
                 b.loadURIWithFlags(aURI, flags, aReferrerURI, aCharset, aPostData);
-
+              } catch (ex) {
+                Cu.reportError(ex);
               }
-              catch (ex) { }
             }
 
             // We start our browsers out as inactive, and then maintain
             // activeness in the tab switcher.
             b.docShell.isActive = false;
 
             // Check if we're opening a tab related to the current tab and
             // move it to after the current tab.