Bug 1140638 - Disable <meta referrer> in Firefox 37 pending some loose ends being sorted out. r=sstamm, a=lmandel
authorBoris Zbarsky <bzbarsky@mit.edu>
Thu, 19 Feb 2015 09:39:16 -0500
changeset 250314 173e1bbcd0b6
parent 250313 46aa1d78bd2e
child 250315 a982b8eabc42
push id4544
push userryanvm@gmail.com
push date2015-03-09 19:49 +0000
treeherdermozilla-beta@d59086707825 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssstamm, lmandel
bugs1140638
milestone37.0
Bug 1140638 - Disable <meta referrer> in Firefox 37 pending some loose ends being sorted out. r=sstamm, a=lmandel
dom/base/nsDocument.cpp
dom/base/test/mochitest.ini
parser/html/nsHtml5TreeOpExecutor.cpp
--- a/dom/base/nsDocument.cpp
+++ b/dom/base/nsDocument.cpp
@@ -3788,17 +3788,18 @@ nsDocument::SetHeaderData(nsIAtom* aHead
       aHeaderField == nsGkAtoms::viewport_initial_scale ||
       aHeaderField == nsGkAtoms::viewport_height ||
       aHeaderField == nsGkAtoms::viewport_width ||
       aHeaderField ==  nsGkAtoms::viewport_user_scalable) {
     mViewportType = Unknown;
   }
 
   // Referrer policy spec says to ignore any empty referrer policies.
-  if (aHeaderField == nsGkAtoms::referrer && !aData.IsEmpty()) {
+  // Disabled for now.
+  if (false && aHeaderField == nsGkAtoms::referrer && !aData.IsEmpty()) {
     ReferrerPolicy policy = mozilla::net::ReferrerPolicyFromString(aData);
 
     // Referrer policy spec (section 6.1) says that once the referrer policy
     // is set, any future attempts to change it result in No-Referrer.
     if (!mReferrerPolicySet) {
       mReferrerPolicy = policy;
       mReferrerPolicySet = true;
     } else if (mReferrerPolicy != policy) {
--- a/dom/base/test/mochitest.ini
+++ b/dom/base/test/mochitest.ini
@@ -599,19 +599,21 @@ skip-if = buildapp == 'b2g'
 [test_bug693875.html]
 [test_bug694754.xhtml]
 [test_bug696301-1.html]
 [test_bug696301-2.html]
 [test_bug698381.html]
 [test_bug698384.html]
 [test_bug704063.html]
 [test_bug704320.html]
-skip-if = buildapp == 'b2g' || toolkit == 'android' || e10s # b2g (Needs multiple window.open support) android(times out, bug 1100609) e10s(randomly fails, bug 1100362)
+skip-if = true
 [test_bug704320_policyset.html]
+skip-if = true
 [test_bug704320_preload.html]
+skip-if = true
 [test_bug707142.html]
 [test_bug708620.html]
 [test_bug711047.html]
 [test_bug711180.html]
 [test_bug719533.html]
 [test_bug726364.html]
 [test_bug737087.html]
 [test_bug737565.html]
--- a/parser/html/nsHtml5TreeOpExecutor.cpp
+++ b/parser/html/nsHtml5TreeOpExecutor.cpp
@@ -954,16 +954,18 @@ nsHtml5TreeOpExecutor::SetSpeculationBas
   DebugOnly<nsresult> rv = NS_NewURI(getter_AddRefs(mSpeculationBaseURI), aURL,
                                      charset.get(), mDocument->GetDocumentURI());
   NS_WARN_IF_FALSE(NS_SUCCEEDED(rv), "Failed to create a URI");
 }
 
 void
 nsHtml5TreeOpExecutor::SetSpeculationReferrerPolicy(const nsAString& aReferrerPolicy)
 {
+  // Disabled for now.
+  return;
   ReferrerPolicy policy = mozilla::net::ReferrerPolicyFromString(aReferrerPolicy);
   return SetSpeculationReferrerPolicy(policy);
 }
 
 void
 nsHtml5TreeOpExecutor::SetSpeculationReferrerPolicy(ReferrerPolicy aReferrerPolicy)
 {
   if (mSpeculationReferrerPolicyWasSet &&