Changed EPosition to an enum class and renamed its members

third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 330 matching lines @@
       else
         toLayoutBox(child)->unmarkOrthogonalWritingModeRoot();
     }
   }
 
   // Fixed-position is painted using transform. In the case that the object
   // gets the same layout after changing position property, although no
   // re-raster (rect-based invalidation) is needed, display items should
   // still update their paint offset.
   if (oldStyle) {
-    bool newStyleIsFixedPosition = style()->position() == FixedPosition;
-    bool oldStyleIsFixedPosition = oldStyle->position() == FixedPosition;
+    bool newStyleIsFixedPosition = style()->position() == EPosition::kFixed;
+    bool oldStyleIsFixedPosition = oldStyle->position() == EPosition::kFixed;
     if (newStyleIsFixedPosition != oldStyleIsFixedPosition)
       ObjectPaintInvalidator(*this)
           .invalidateDisplayItemClientsIncludingNonCompositingDescendants(
               PaintInvalidationStyleChange);
   }
 
   // The used style for body background may change due to computed style change
   // on the document element because of background stealing.
   // Refer to backgroundStolenForBeingBody() and
   // http://www.w3.org/TR/css3-background/#body-background for more info.
   if (isDocumentElement()) {
     HTMLBodyElement* body = document().firstBodyElement();
     LayoutObject* bodyLayout = body ? body->layoutObject() : nullptr;
     if (bodyLayout && bodyLayout->isBoxModelObject()) {
       bool newStoleBodyBackground = toLayoutBoxModelObject(bodyLayout)
                                         ->backgroundStolenForBeingBody(style());
       bool oldStoleBodyBackground =
           oldStyle &&
           toLayoutBoxModelObject(bodyLayout)
               ->backgroundStolenForBeingBody(oldStyle);
       if (newStoleBodyBackground != oldStoleBodyBackground &&
           bodyLayout->style() && bodyLayout->style()->hasBackground()) {
         bodyLayout->setShouldDoFullPaintInvalidation();
       }
     }
   }
 
   if (FrameView* frameView = view()->frameView()) {
-    bool newStyleIsViewportConstained = style()->position() == FixedPosition;
+    bool newStyleIsViewportConstained =
+        style()->position() == EPosition::kFixed;
     bool oldStyleIsViewportConstrained =
-        oldStyle && oldStyle->position() == FixedPosition;
-    bool newStyleIsSticky = style()->position() == StickyPosition;
-    bool oldStyleIsSticky = oldStyle && oldStyle->position() == StickyPosition;
+        oldStyle && oldStyle->position() == EPosition::kFixed;
+    bool newStyleIsSticky = style()->position() == EPosition::kSticky;
+    bool oldStyleIsSticky =
+        oldStyle && oldStyle->position() == EPosition::kSticky;
 
     if (newStyleIsSticky != oldStyleIsSticky) {
       if (newStyleIsSticky) {
         // During compositing inputs update we'll have the scroll ancestor
         // without having to walk up the tree and can compute the sticky
         // position constraints then.
         if (layer())
           layer()->setNeedsCompositingInputsUpdate();
 
         // TODO(pdr): When slimming paint v2 is enabled, we will need to
@@ skipping 642 matching lines @@
       if (offsetParentObject->isBox() && offsetParentObject->isBody() &&
           !offsetParentObject->isPositioned()) {
         referencePoint.moveBy(
             toLayoutBox(offsetParentObject)->physicalLocation());
       }
     }
 
     if (offsetParentObject->isLayoutInline()) {
       const LayoutInline* inlineParent = toLayoutInline(offsetParentObject);
 
-      if (isBox() && style()->position() == AbsolutePosition &&
+      if (isBox() && style()->position() == EPosition::kAbsolute &&
           inlineParent->isInFlowPositioned()) {
         // Offset for absolute elements with inline parent is a special
         // case in the CSS spec
         referencePoint +=
             inlineParent->offsetForInFlowPositionedInline(*toLayoutBox(this));
       }
 
       referencePoint -= inlineParent->firstLineBoxTopLeft();
     }
 
@@ skipping 160 matching lines @@
     const LayoutBoxModelObject* ancestorToStopAt,
     LayoutGeometryMap& geometryMap) const {
   ASSERT(ancestorToStopAt != this);
 
   AncestorSkipInfo skipInfo(ancestorToStopAt);
   LayoutObject* container = this->container(&skipInfo);
   if (!container)
     return nullptr;
 
   bool isInline = isLayoutInline();
-  bool isFixedPos = !isInline && style()->position() == FixedPosition;
+  bool isFixedPos = !isInline && style()->position() == EPosition::kFixed;
   bool containsFixedPosition = canContainFixedPositionObjects();
 
   LayoutSize adjustmentForSkippedAncestor;
   if (skipInfo.ancestorSkipped()) {
     // There can't be a transform between paintInvalidationContainer and
     // ancestorToStopAt, because transforms create containers, so it should be
     // safe to just subtract the delta between the ancestor and
     // ancestorToStopAt.
     adjustmentForSkippedAncestor =
         -ancestorToStopAt->offsetFromAncestorContainer(container);
@@ skipping 119 matching lines @@
   if (rootElementStyle->hasBackground())
     return false;
 
   if (node() != document().firstBodyElement())
     return false;
 
   return true;
 }
 
 }  // namespace blink