Bug 1429979 Part 2 - Use ranged-based for loop to iterate FrameSet.
MozReview-Commit-ID: I1aP3Kpqebf
--- a/layout/style/ImageLoader.cpp
+++ b/layout/style/ImageLoader.cpp
@@ -354,20 +354,17 @@ void InvalidateImages(nsIFrame* aFrame)
}
void
ImageLoader::DoRedraw(FrameSet* aFrameSet, bool aForcePaint)
{
NS_ASSERTION(aFrameSet, "Must have a frame set");
NS_ASSERTION(mDocument, "Should have returned earlier!");
- FrameSet::size_type length = aFrameSet->Length();
- for (FrameSet::size_type i = 0; i < length; i++) {
- nsIFrame* frame = aFrameSet->ElementAt(i);
-
+ for (nsIFrame* frame : *aFrameSet) {
if (frame->StyleVisibility()->IsVisible()) {
if (frame->IsFrameOfType(nsIFrame::eTablePart)) {
// Tables don't necessarily build border/background display items
// for the individual table part frames, so IterateRetainedDataFor
// might not find the right display item.
frame->InvalidateFrame();
} else {
InvalidateImages(frame);
@@ -436,20 +433,17 @@ ImageLoader::OnSizeAvailable(imgIRequest
aImage->SetAnimationMode(presContext->ImageAnimationMode());
FrameSet* frameSet = nullptr;
if (!mRequestToFrameMap.Get(aRequest, &frameSet)) {
return NS_OK;
}
- FrameSet::size_type length = frameSet->Length();
- for (FrameSet::size_type i = 0; i < length; i++) {
- nsIFrame* frame = frameSet->ElementAt(i);
-
+ for (nsIFrame* frame : *frameSet) {
if (frame->StyleVisibility()->IsVisible()) {
frame->MarkNeedsDisplayItemRebuild();
}
}
return NS_OK;
}