Relayout RenderObjects with viewport-percentage CSS properties when viewport size changes

Source/core/rendering/RenderObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 2042 matching lines @@
 static inline bool areCursorsEqual(const RenderStyle* a, const RenderStyle* b)
 {
     return a->cursor() == b->cursor() && (a->cursors() == b->cursors() || areNonIdenticalCursorListsEqual(a, b));
 }
 
 void RenderObject::styleDidChange(StyleDifference diff, const RenderStyle* oldStyle)
 {
     if (s_affectsParentBlock)
         handleDynamicFloatPositionChange();
 
+    if (RenderView* view = this->view()) {

[esprehn, 2013/10/25 22:25:51]

You don't need to null check this, going through styleDidChange when there's no
view should not be possible

+        bool newStyleMayHaveViewportPercentage = style()->mayHaveViewportPercentageProperty();
+        bool oldStyleMayHaveViewportPercentage = oldStyle && oldStyle->mayHaveViewportPercentageProperty();
+        if (newStyleMayHaveViewportPercentage != oldStyleMayHaveViewportPercentage) {
+            if (newStyleMayHaveViewportPercentage)
+                view->addViewportPercentageStyleObject(this);
+            else
+                view->removeViewportPercentageStyleObject(this);
+        }
+    }
+
     if (!m_parent)
         return;
 
     if (diff == StyleDifferenceLayout || diff == StyleDifferenceSimplifiedLayout) {
         RenderCounter::rendererStyleChanged(this, oldStyle, m_style.get());
 
         // If the object already needs layout, then setNeedsLayout won't do
         // any work. But if the containing block has changed, then we may need
         // to mark the new containing blocks for layout. The change that can
         // directly affect the containing block of this object is a change to
@@ skipping 529 matching lines @@
     // FIXME: Would like to do this in RenderBoxModelObject, but the timing is so complicated that this can't easily
     // be moved into RenderBoxModelObject::destroy.
     if (hasLayer()) {
         setHasLayer(false);
         toRenderLayerModelObject(this)->destroyLayer();
     }
 
     setAncestorLineBoxDirty(false);
 
     clearLayoutRootIfNeeded();
+
+    if (RenderView* view = this->view())
+        view->removeViewportPercentageStyleObject(this);
 }
 
 void RenderObject::insertedIntoTree()
 {
     // FIXME: We should ASSERT(isRooted()) here but generated content makes some out-of-order insertion.
 
     // Keep our layer hierarchy updated. Optimize for the common case where we don't have any children
     // and don't have a layer attached to ourselves.
     RenderLayer* layer = 0;
     if (firstChild() || hasLayer()) {
@@ skipping 773 matching lines @@
 {
     if (object1) {
         const WebCore::RenderObject* root = object1;
         while (root->parent())
             root = root->parent();
         root->showRenderTreeAndMark(object1, "*", object2, "-", 0);
     }
 }
 
 #endif