Don't try to reframe generated content for image state changes on it. Bug

313656, r=biesi, sr=dbaron

content/base/src/nsGenConImageContent.cpp

@@ -101,5 +101,6 @@ nsGenConImageContent::IntrinsicState() const
     imageState |= NS_EVENT_STATE_SUPPRESSED;
     imageState &= ~NS_EVENT_STATE_BROKEN;
   }
+  imageState &= ~NS_EVENT_STATE_LOADING;
   return state | imageState;
 }

layout/base/nsCSSFrameConstructor.cpp

@@ -10614,8 +10614,10 @@ nsCSSFrameConstructor::DoContentStateChanged(nsIContent* aContent,
     // call will handle things.
     nsIFrame* primaryFrame = mPresShell->GetPrimaryFrameFor(aContent);
     if (primaryFrame) {
-      if (aStateMask & (NS_EVENT_STATE_BROKEN | NS_EVENT_STATE_USERDISABLED |
-                        NS_EVENT_STATE_SUPPRESSED | NS_EVENT_STATE_LOADING)) {
+      // If it's generated content, ignore LOADING/etc state changes on it.
+      if (!primaryFrame->IsGeneratedContentFrame() &&
+          (aStateMask & (NS_EVENT_STATE_BROKEN | NS_EVENT_STATE_USERDISABLED |
+                         NS_EVENT_STATE_SUPPRESSED | NS_EVENT_STATE_LOADING))) {
         hint = nsChangeHint_ReconstructFrame;
       } else {          
         PRUint8 app = primaryFrame->GetStyleDisplay()->mAppearance;