Bug 1377923 - Don't check if observer service is found in image loading/showing reporting code. r=jwatt
MozReview-Commit-ID: Db7rkiAhUVq
--- a/image/ImageFactory.cpp
+++ b/image/ImageFactory.cpp
@@ -91,21 +91,19 @@ ImageFactory::CreateImage(nsIRequest* aR
// Compute the image's initialization flags.
uint32_t imageFlags = ComputeImageFlags(aURI, aMimeType, aIsMultiPart);
#ifdef DEBUG
// Record the image load for startup performance testing.
if (NS_IsMainThread()) {
nsCOMPtr<nsIObserverService> obs = services::GetObserverService();
- if (NS_WARN_IF(obs)) {
- nsAutoCString spec;
- aURI->GetSpec(spec);
- obs->NotifyObservers(nullptr, "image-loading", NS_ConvertUTF8toUTF16(spec).get());
- }
+ nsAutoCString spec;
+ aURI->GetSpec(spec);
+ obs->NotifyObservers(nullptr, "image-loading", NS_ConvertUTF8toUTF16(spec).get());
}
#endif
// Select the type of image to create based on MIME type.
if (aMimeType.EqualsLiteral(IMAGE_SVG_XML)) {
return CreateVectorImage(aRequest, aProgressTracker, aMimeType,
aURI, imageFlags, aInnerWindowId);
} else {
--- a/image/RasterImage.cpp
+++ b/image/RasterImage.cpp
@@ -1394,22 +1394,20 @@ RasterImage::DrawInternal(DrawableSurfac
gfxContextMatrixAutoSaveRestore saveMatrix(aContext);
ImageRegion region(aRegion);
bool frameIsFinished = aSurface->IsFinished();
#ifdef DEBUG
// Record the image drawing for startup performance testing.
if (NS_IsMainThread()) {
nsCOMPtr<nsIObserverService> obs = services::GetObserverService();
- if (NS_WARN_IF(obs)) {
- nsCOMPtr<nsIURI> imageURI = mURI->ToIURI();
- nsAutoCString spec;
- imageURI->GetSpec(spec);
- obs->NotifyObservers(nullptr, "image-drawing", NS_ConvertUTF8toUTF16(spec).get());
- }
+ nsCOMPtr<nsIURI> imageURI = mURI->ToIURI();
+ nsAutoCString spec;
+ imageURI->GetSpec(spec);
+ obs->NotifyObservers(nullptr, "image-drawing", NS_ConvertUTF8toUTF16(spec).get());
}
#endif
// By now we may have a frame with the requested size. If not, we need to
// adjust the drawing parameters accordingly.
IntSize finalSize = aSurface->GetImageSize();
bool couldRedecodeForBetterFrame = false;
if (finalSize != aSize) {
--- a/image/VectorImage.cpp
+++ b/image/VectorImage.cpp
@@ -1024,22 +1024,20 @@ VectorImage::Show(gfxDrawable* aDrawable
SurfaceFormat::B8G8R8A8,
aParams.samplingFilter,
aParams.flags, aParams.opacity);
#ifdef DEBUG
// Record the image drawing for startup performance testing.
if (NS_IsMainThread()) {
nsCOMPtr<nsIObserverService> obs = services::GetObserverService();
- if (NS_WARN_IF(obs)) {
- nsCOMPtr<nsIURI> imageURI = mURI->ToIURI();
- nsAutoCString spec;
- imageURI->GetSpec(spec);
- obs->NotifyObservers(nullptr, "image-drawing", NS_ConvertUTF8toUTF16(spec).get());
- }
+ nsCOMPtr<nsIURI> imageURI = mURI->ToIURI();
+ nsAutoCString spec;
+ imageURI->GetSpec(spec);
+ obs->NotifyObservers(nullptr, "image-drawing", NS_ConvertUTF8toUTF16(spec).get());
}
#endif
MOZ_ASSERT(mRenderingObserver, "Should have a rendering observer by now");
mRenderingObserver->ResumeHonoringInvalidations();
}
void