Bug 1134606 - Disable <meta referrer> in Firefox 36 pending some loose ends being sorted out. r=sstamm, a=sledru
authorBoris Zbarsky <bzbarsky@mit.edu>
Thu, 19 Feb 2015 09:39:16 -0500
changeset 245413 521cf86d194b
parent 245412 1a2baaf50371
child 245414 89cfa8ff9fc5
push id661
push userryanvm@gmail.com
push date2015-02-19 21:33 +0000
treeherdermozilla-release@2f2abd6ffebb [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssstamm, sledru
bugs1134606
milestone36.0
Bug 1134606 - Disable <meta referrer> in Firefox 36 pending some loose ends being sorted out. r=sstamm, a=sledru
dom/base/nsDocument.cpp
dom/base/test/mochitest.ini
parser/html/nsHtml5TreeOpExecutor.cpp
--- a/dom/base/nsDocument.cpp
+++ b/dom/base/nsDocument.cpp
@@ -3729,17 +3729,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
@@ -595,19 +595,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);
 
   if (mSpeculationReferrerPolicyWasSet &&
       policy != mSpeculationReferrerPolicy) {
     // According to the Referrer Policy spec, if there's already been a policy
     // set and another attempt is made to set a _different_ policy, the result
     // is a "No Referrer" policy.
     mSpeculationReferrerPolicy = mozilla::net::RP_No_Referrer;