Bug 836176 - Part 2: Remove nsIHTMLDocument::SetIsXHTML(); r=farre

Differential Revision: https://phabricator.services.mozilla.com/D32924

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Ehsan Akhgari 2019-06-04 17:27:41 +00:00
parent 2289134635
commit b952303dbe
3 changed files with 3 additions and 9 deletions

View File

@ -85,9 +85,7 @@ class nsHTMLDocument : public mozilla::dom::Document, public nsIHTMLDocument {
virtual void AddedForm() override;
virtual void RemovedForm() override;
virtual int32_t GetNumFormsSynchronous() override;
virtual void SetIsXHTML(bool aXHTML) override {
mType = (aXHTML ? eXHTML : eHTML);
}
void SetIsXHTML(bool aXHTML) { mType = (aXHTML ? eXHTML : eHTML); }
virtual nsresult Clone(mozilla::dom::NodeInfo*,
nsINode** aResult) const override;

View File

@ -44,8 +44,6 @@ class nsIHTMLDocument : public nsISupports {
// XXXbz is this still needed now that we can flush just content,
// not the rest?
virtual int32_t GetNumFormsSynchronous() = 0;
virtual void SetIsXHTML(bool aXHTML) = 0;
};
NS_DEFINE_STATIC_IID_ACCESSOR(nsIHTMLDocument, NS_IHTMLDOCUMENT_IID)

View File

@ -40,7 +40,7 @@
#include "nsNodeUtils.h"
#include "nsIConsoleService.h"
#include "nsIScriptError.h"
#include "nsIHTMLDocument.h"
#include "nsHTMLDocument.h"
#include "mozilla/BasicEvents.h"
#include "mozilla/EventDispatcher.h"
#include "mozilla/Encoding.h"
@ -121,10 +121,8 @@ nsresult NS_NewDOMDocument(Document** aInstancePtrResult,
}
if (isHTML) {
nsCOMPtr<nsIHTMLDocument> htmlDoc = do_QueryInterface(d);
NS_ASSERTION(htmlDoc, "HTML Document doesn't implement nsIHTMLDocument?");
d->SetCompatibilityMode(eCompatibility_FullStandards);
htmlDoc->SetIsXHTML(isXHTML);
d->AsHTMLDocument()->SetIsXHTML(isXHTML);
}
d->SetLoadedAsData(aLoadedAsData);
d->SetDocumentURI(aDocumentURI);