Created attachment 338954 [details] [diff] [review] cleanup This is one of the tasks listed on https://wiki.mozilla.org/Gecko:DeCOMtamination I moved nsNameSpaceManagerImpl into nsINameSpaceManager, got rid of virtual methods. Not sure what else is one is to do. It's pretty trivial and none of these methods are outparamdel-able. Should change the class to not inherit from nsISupports and s/nsCOMPtr/nsRefPtr/ for pointers containing it?
Created attachment 338961 [details] [diff] [review] cleanup accidentally sent a stale ver
Comment on attachment 338961 [details] [diff] [review] cleanup class nsINameSpaceManager : public nsISupports This could totally be a followup bug, but there's no real need to inherit nsISupports here any longer I don't think. And this class should be nsNameSpaceManager now, no need for ns_I_ here any more IMO. + nsresult RegisterNameSpace(const nsAString& aURI, + PRInt32& aNameSpaceID); Got some auto-emacs-indentation magic pixie dust that would help here too? If not, fix this manually before landing :) +nsINameSpaceManager::RegisterNameSpace(const nsAString& aURI, PRInt32& aNameSpaceID) Here, +nsresult nsINameSpaceManager::AddNameSpace(const nsAString& aURI, const PRInt32 aNameSpaceID) and here too. r+sr=jst with that.
Assignee: nobody → tglek
QA Contact: content
Hardware: PC → All
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
This broke up the windows and linux leak test builds, so backed out.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
To give you more of a hint than that, it fried everything building both accessibility and -enable-shared: SeaMonkey builds everything shared, so all their Linux and Windows builds, Thunderbird only builds unittests shared, so Windows there (and would have been Linux, except it was already busted). I wondered if it just needed a clobber, so depending on when it started, we may or may not have gotten a clobbered Tb Windows check build before the backout.
Nope, backout made it in before my clobber started.
Don't forget debug builds are also shared by default.
Yeah, so with this patch I get: Undefined symbols: "nsINameSpaceManager::GetNameSpaceURI(int, nsAString_internal&)", referenced from: nsDocAccessible::GetNameSpaceURIForID(short, nsAString_internal&) in nsDocAccessible.o because that function became non-virtual..
jst, is this still a candidate for decomtamination? Once I rename and remove nsISupports, are the downstream projects expected to adapt?
Is it still possible to deCOM this or should we just close the bug?
Created attachment 606884 [details] [diff] [review] WIP I don't remember why I stopped working on this bug; this seems to be where I got.
You need to log in before you can comment on or make changes to this bug.