Before bug 968805, none of the callers would reach this branch. The callers in
nsContentDLF.cpp call LoadStart immediately after constructing the
nsDocumentViewer, so they hit the !mDocument case, and the comparison was
always false for the caller in nsHTMLDocument::Open, as it passed an
nsIHTMLDocument pointer, so the nsISupports pointers being compared were
always different. (XPCOM rules require SameCOMIdentity in this case, to avoid
exactly this issue.)
I tested that the removed code was never reached after a backout of bug 968805.
This change only impacts developer builds. A developer build is one
where MOZILLA_OFFICIAL is not set.
The startup cache is invalidated when the buildid changes; see [1] for
details. For MOZILLA_OFFICIAL builds, the buildid is always bumped, so
the startup cache is always fresh on a package re-deploy to device.
Most developers re-deploy using |mach build mobile/android && mach
package && mach install| or similar. This does not bump the buildid.
The re-deployed package will read the out-dated startup cache, leading
to frustrating inconsistencies when developing Javascript, especially
chrome content and module JSMs.
This change purges the startup caches every time Gecko is started in
developer builds. This keeps the running Javascript consistent (which
is good for development), but incurs a startup performance
penalty (since the cache must be purged, and since the cached files must
be recompiled, etc).
[1] http://hg.mozilla.org/mozilla-central/file/901d300bb441/toolkit/xre/nsAppRunner.cpp#l2350
========
https://hg.mozilla.org/integration/gaia-central/rev/557d2a3432f0
Author: Gregor Wagner <anygregor@gmail.com>
Desc: Merge pull request #18901 from ckerschb/cspTestContainerFix
Bug 858787 - Loading css for test-container from external file. r=fabrice
========
https://hg.mozilla.org/integration/gaia-central/rev/35723984b298
Author: Christoph Kerschbaumer <mozilla@christophkerschbaumer.com>
Desc: Flipping pref csp.speccompliant causes tests to fail on the B2G Emulator,
because test-container/index.html uses an inline-style which is blocked by CSP.
Rewrite of the *.html file to load the css/style information from an external file
CSP does not block it.