Backed out changeset 19fbc34f239c (bug 1374629)

MozReview-Commit-ID: KPYcFSnOpU8

Author: KWierso

dom/base/FormData.cpp

@@ -18,7 +18,7 @@ using namespace mozilla;
 using namespace mozilla::dom;
 
 FormData::FormData(nsISupports* aOwner)
-  : HTMLFormSubmission(UTF_8_ENCODING, nullptr)
+  : HTMLFormSubmission(WrapNotNull(UTF_8_ENCODING), nullptr)
   , mOwner(aOwner)
 {
 }
@@ -402,7 +402,7 @@ NS_IMETHODIMP
 FormData::GetSendInfo(nsIInputStream** aBody, uint64_t* aContentLength,
                       nsACString& aContentTypeWithCharset, nsACString& aCharset)
 {
-  FSMultipartFormData fs(UTF_8_ENCODING, nullptr);
+  FSMultipartFormData fs(WrapNotNull(UTF_8_ENCODING), nullptr);
 
   for (uint32_t i = 0; i < mFormData.Length(); ++i) {
     if (mFormData[i].wasNullBlob) {

dom/html/HTMLFormSubmission.cpp

@@ -889,7 +889,7 @@ GetSubmitEncoding(nsGenericHTMLElement* aForm)
   if (doc) {
     return Encoding::ForName(doc->GetDocumentCharacterSet());
   }
-  return UTF_8_ENCODING;
+  return WrapNotNull(UTF_8_ENCODING);
 }
 
 void

intl/Encoding.h

@@ -27,7 +27,6 @@ class Encoder;
 }; // namespace mozilla
 
 #define ENCODING_RS_ENCODING mozilla::Encoding
-#define ENCODING_RS_NOT_NULL_CONST_ENCODING_PTR mozilla::NotNull<const mozilla::Encoding*>
 #define ENCODING_RS_ENCODER mozilla::Encoder
 #define ENCODING_RS_DECODER mozilla::Decoder
 

intl/gtest/TestEncoding.cpp

@@ -4,17 +4,11 @@
 #include "gtest/gtest.h"
 
 #include "mozilla/Encoding.h"
-#include <type_traits>
 
 #define ENCODING_TEST(name) TEST(EncodingTest, name)
 
 using namespace mozilla;
 
-static_assert(std::is_trivially_copyable<NotNull<const Encoding*>>::value,
-              "NotNull<const Encoding*> must be trivially copyable.");
-static_assert(std::is_standard_layout<NotNull<const Encoding*>>::value,
-              "NotNull<const Encoding*> must be a standard layout type.");
-
 // These tests mainly test that the C++ interface seems to
 // reach the Rust code. More thorough testing of the back
 // end is done in Rust.