Rename rendering/RenderList* to layout/LayoutList*

Rename rendering/RenderList* to layout/LayoutList*

BUG=450612
R=dsinclair@chromium.org,pdr@chromium.org,tkent@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190731

[bsittler, 2015-02-19 00:21:47]

The CQ bit was checked by bsittler@chromium.org

[bsittler, 2015-02-19 00:21:57]

This change renames RenderList* to LayoutList* and moves their files from
rendering/ to layout/.

[commit-bot: I haz the power, 2015-02-19 03:25:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/931423003/1

[commit-bot: I haz the power, 2015-02-19 03:25:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-19 03:25:38]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[dsinclair, 2015-02-19 15:49:23]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2015-02-19 15:49:30]

lgtm

[commit-bot: I haz the power, 2015-02-19 15:51:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/931423003/1

[commit-bot: I haz the power, 2015-02-19 16:10:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-19 16:10:55]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/43658)

[bsittler, 2015-02-20 01:30:56]

New patchsets have been uploaded after l-g-t-m from dsinclair@chromium.org

[bsittler, 2015-02-23 23:00:06]

The CQ bit was checked by bsittler@chromium.org

[bsittler, 2015-02-23 23:00:10]

The patchset sent to the CQ was uploaded after l-g-t-m from
dsinclair@chromium.org
Link to the patchset: https://codereview.chromium.org/931423003/#ps20001 (title:
"Resolve conflicts from recent renamings and rename stray member variables.")

[commit-bot: I haz the power, 2015-02-23 23:01:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/931423003/20001

[bsittler, 2015-02-23 23:27:08]

bsittler@chromium.org changed reviewers:
+ tkent@chromium.org

[bsittler, 2015-02-24 00:37:08]

bsittler@chromium.org changed reviewers:
+ pdr@chromium.org

[bsittler, 2015-02-24 00:37:12]

PTAL; resolved conflicts with recent changes; needs OWNERS review.

[pdr., 2015-02-24 01:09:25]

On 2015/02/24 at 00:37:12, bsittler wrote:
> PTAL; resolved conflicts with recent changes; needs OWNERS review.

LGTM RS=me

[bsittler, 2015-02-24 01:55:15]

The CQ bit was checked by bsittler@chromium.org

[bsittler, 2015-02-24 01:55:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
dsinclair@chromium.org
Link to the patchset: https://codereview.chromium.org/931423003/#ps40001 (title:
"Resolve conflicts with recent renamings.")

[commit-bot: I haz the power, 2015-02-24 01:56:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/931423003/40001

[commit-bot: I haz the power, 2015-02-24 04:34:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-24 04:35:06]

Failed to apply patch for Source/core/layout/LayoutListMarker.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  A         Source/core/layout/LayoutListMarker.cpp
  Copied Source/core/rendering/RenderListMarker.cpp ->
Source/core/layout/LayoutListMarker.cpp
  patching file Source/core/layout/LayoutListMarker.cpp
  Hunk #21 FAILED at 1616.
  1 out of 22 hunks FAILED -- saving rejects to file
Source/core/layout/LayoutListMarker.cpp.rej

Patch:   NR 
Source/core/rendering/RenderListMarker.cpp->Source/core/layout/LayoutListMarker.cpp
Index: Source/core/layout/LayoutListMarker.cpp
diff --git a/Source/core/rendering/RenderListMarker.cpp
b/Source/core/layout/LayoutListMarker.cpp
similarity index 48%
rename from Source/core/rendering/RenderListMarker.cpp
rename to Source/core/layout/LayoutListMarker.cpp
index
648c067252323e748873c42ccaa7920a0b3a0789..f82e7c9e920c5403bb194d2d2436c24ef07e8548
100644
--- a/Source/core/rendering/RenderListMarker.cpp
+++ b/Source/core/layout/LayoutListMarker.cpp
@@ -23,13 +23,13 @@
  */
 
 #include "config.h"
-#include "core/rendering/RenderListMarker.h"
+#include "core/layout/LayoutListMarker.h"
 
 #include "core/fetch/ImageResource.h"
 #include "core/layout/Layer.h"
+#include "core/layout/LayoutListItem.h"
 #include "core/layout/TextRunConstructor.h"
 #include "core/paint/ListMarkerPainter.h"
-#include "core/rendering/RenderListItem.h"
 #include "platform/fonts/Font.h"
 #include "wtf/text/StringBuilder.h"
 
@@ -205,9 +205,9 @@ static String toHebrew(int number)
     UChar letters[lettersSize];
 
     int length;
-    if (number < 1000)
+    if (number < 1000) {
         length = 0;
-    else {
+    } else {
         length = toHebrewUnder1000(number / 1000, letters);
         letters[length++] = '\'';
         number = number % 1000;
@@ -323,19 +323,19 @@ static String toCJKIdeographic(int number, const UChar
table[16])
     ASSERT(number >= 0);
 
     enum AbstractCJKChar {
-        noChar,
-        secondGroupMarker, thirdGroupMarker, fourthGroupMarker,
-        secondDigitMarker, thirdDigitMarker, fourthDigitMarker,
-        digit0, digit1, digit2, digit3, digit4,
-        digit5, digit6, digit7, digit8, digit9
+        NoChar,
+        SecondGroupMarker, ThirdGroupMarker, FourthGroupMarker,
+        SecondDigitMarker, ThirdDigitMarker, FourthDigitMarker,
+        Digit0, Digit1, Digit2, Digit3, Digit4,
+        Digit5, Digit6, Digit7, Digit8, Digit9
     };
 
     if (number == 0)
-        return String(&table[digit0 - 1], 1);
+        return String(&table[Digit0 - 1], 1);
 
     const int groupLength = 8; // 4 digits, 3 digit markers, and a group marker
     const int bufferLength = 4 * groupLength;
-    AbstractCJKChar buffer[bufferLength] = { noChar };
+    AbstractCJKChar buffer[bufferLength] = { NoChar };
 
     for (int i = 0; i < 4; ++i) {
         int groupValue = number % 10000;
@@ -345,54 +345,54 @@ static String toCJKIdeographic(int number, const UChar
table[16])
         AbstractCJKChar* group = &buffer[(3 - i) * groupLength];
 
         if (groupValue && i)
-            group[7] = static_cast<AbstractCJKChar>(secondGroupMarker - 1 + i);
+            group[7] = static_cast<AbstractCJKChar>(SecondGroupMarker - 1 + i);
 
         // Put in the four digits and digit markers for any non-zero digits.
-        group[6] = static_cast<AbstractCJKChar>(digit0 + (groupValue % 10));
+        group[6] = static_cast<AbstractCJKChar>(Digit0 + (groupValue % 10));
         if (number != 0 || groupValue > 9) {
             int digitValue = ((groupValue / 10) % 10);
-            group[4] = static_cast<AbstractCJKChar>(digit0 + digitValue);
+            group[4] = static_cast<AbstractCJKChar>(Digit0 + digitValue);
             if (digitValue)
-                group[5] = secondDigitMarker;
+                group[5] = SecondDigitMarker;
         }
         if (number != 0 || groupValue > 99) {
             int digitValue = ((groupValue / 100) % 10);
-            group[2] = static_cast<AbstractCJKChar>(digit0 + digitValue);
+            group[2] = static_cast<AbstractCJKChar>(Digit0 + digitValue);
             if (digitValue)
-                group[3] = thirdDigitMarker;
+                group[3] = ThirdDigitMarker;
         }
         if (number != 0 || groupValue > 999) {
             int digitValue = groupValue / 1000;
-            group[0] = static_cast<AbstractCJKChar>(digit0 + digitValue);
+            group[0] = static_cast<AbstractCJKChar>(Digit0 + digitValue);
             if (digitValue)
-                group[1] = fourthDigitMarker;
+                group[1] = FourthDigitMarker;
         }
 
         // Remove the tens digit, but leave the marker, for any group that has
         // a value of less than 20.
         if (groupValue < 20) {
-            ASSERT(group[4] == noChar || group[4] == digit0 || group[4] ==
digit1);
-            group[4] = noChar;
+            ASSERT(group[4] == NoChar || group[4] == Digit0 || group[4] ==
Digit1);
+            group[4] = NoChar;
         }
 
         if (number == 0)
             break;
     }
 
-    // Convert into characters, omitting consecutive runs of digit0 and
-    // any trailing digit0.
+    // Convert into characters, omitting consecutive runs of Digit0 and
+    // any trailing Digit0.
     int length = 0;
     UChar characters[bufferLength];
-    AbstractCJKChar last = noChar;
+    AbstractCJKChar last = NoChar;
     for (int i = 0; i < bufferLength; ++i) {
         AbstractCJKChar a = buffer[i];
-        if (a != noChar) {
-            if (a != digit0 || last != digit0)
+        if (a != NoChar) {
+            if (a != Digit0 || last != Digit0)
                 characters[length++] = table[a - 1];
             last = a;
         }
     }
-    if (last == digit0)
+    if (last == Digit0)
         --length;
 
     return String(characters, length);
@@ -497,7 +497,7 @@ static EListStyleType effectiveListMarkerType(EListStyleType
type, int value)
     return type;
 }
 
-UChar RenderListMarker::listMarkerSuffix(EListStyleType type, int value)
+UChar LayoutListMarker::listMarkerSuffix(EListStyleType type, int value)
 {
     // If the list-style-type cannot represent |value| because it's outside its
     // ordinal range then we fall back to some list style that can represent
|value|.
@@ -601,481 +601,481 @@ String listMarkerText(EListStyleType type, int value)
     // If the list-style-type, say hebrew, cannot represent |value| because
it's outside
     // its ordinal range then we fallback to some list style that can represent
|value|.
     switch (effectiveListMarkerType(type, value)) {
-        case NoneListStyle:
-            return "";
-
-        case Asterisks: {
-            static const LChar asterisksSymbols[1] = {
-                0x2A
-            };
-            return toSymbolic(value, asterisksSymbols);
-        }
-        // We use the same characters for text security.
-        // See RenderText::setInternalString.
-        case Circle:
-            return String(&whiteBullet, 1);
-        case Disc:
-            return String(&bullet, 1);
-        case Footnotes: {
-            static const UChar footnotesSymbols[4] = {
-                0x002A, 0x2051, 0x2020, 0x2021
-            };
-            return toSymbolic(value, footnotesSymbols);
-        }
-        case Square:
-            // The CSS 2.1 test suite uses U+25EE BLACK MEDIUM SMALL SQUARE
-            // instead, but I think this looks better.
-            return String(&blackSquare, 1);
+    case NoneListStyle:
+        return "";
+
+    case Asterisks: {
+        static const LChar asterisksSymbols[1] = {
+            0x2A
+        };
+        return toSymbolic(value, asterisksSymbols);
+    }
+    // We use the same characters for text security.
+    // See RenderText::setInternalString.
+    case Circle:
+        return String(&whiteBullet, 1);
+    case Disc:
+        return String(&bullet, 1);
+    case Footnotes: {
+        static const UChar footnotesSymbols[4] = {
+            0x002A, 0x2051, 0x2020, 0x2021
+        };
+        return toSymbolic(value, footnotesSymbols);
+    }
+    case Square:
+        // The CSS 2.1 test suite uses U+25EE BLACK MEDIUM SMALL SQUARE
+        // instead, but I think this looks better.
+        return String(&blackSquare, 1);
 
-        case DecimalListStyle:
+    case DecimalListStyle:
+        return String::number(value);
+    case DecimalLeadingZero:
+        if (value < -9 || value > 9)
             return String::number(value);
-        case DecimalLeadingZero:
-            if (value < -9 || value > 9)
-                return String::number(value);
-            if (value < 0)
-                return "-0" + String::number(-value); // -01 to -09
-            return "0" + String::number(value); // 00 to 09
-
-        case ArabicIndic: {
-            static const UChar arabicIndicNumerals[10] = {
-                0x0660, 0x0661, 0x0662, 0x0663, 0x0664, 0x0665, 0x0666, 0x0667,
0x0668, 0x0669
-            };
-            return toNumeric(value, arabicIndicNumerals);
-        }
-        case BinaryListStyle: {
-            static const LChar binaryNumerals[2] = {
-                '0', '1'
-            };
-            return toNumeric(value, binaryNumerals);
-        }
-        case Bengali: {
-            static const UChar bengaliNumerals[10] = {
-                0x09E6, 0x09E7, 0x09E8, 0x09E9, 0x09EA, 0x09EB, 0x09EC, 0x09ED,
0x09EE, 0x09EF
-            };
-            return toNumeric(value, bengaliNumerals);
-        }
-        case Cambodian:
-        case Khmer: {
-            static const UChar khmerNumerals[10] = {
-                0x17E0, 0x17E1, 0x17E2, 0x17E3, 0x17E4, 0x17E5, 0x17E6, 0x17E7,
0x17E8, 0x17E9
-            };
-            return toNumeric(value, khmerNumerals);
-        }
-        case Devanagari: {
-            static const UChar devanagariNumerals[10] = {
-                0x0966, 0x0967, 0x0968, 0x0969, 0x096A, 0x096B, 0x096C, 0x096D,
0x096E, 0x096F
-  …
(message too large)

[commit-bot: I haz the power, 2015-02-24 04:35:12]

The CQ bit was unchecked by commit-bot@chromium.org

[bsittler, 2015-02-24 06:52:15]

The CQ bit was checked by bsittler@chromium.org

[bsittler, 2015-02-24 06:52:20]

The patchset sent to the CQ was uploaded after l-g-t-m from
dsinclair@chromium.org, pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/931423003/#ps50001 (title:
"Resolve conflicts due to recent changes.")

[commit-bot: I haz the power, 2015-02-24 06:53:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/931423003/50001

[bsittler, 2015-02-24 07:25:45]

The CQ bit was checked by bsittler@chromium.org

[bsittler, 2015-02-24 07:25:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
dsinclair@chromium.org, pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/931423003/#ps60001 (title:
"Remove spurious LayoutLayerModelObject reference")

[commit-bot: I haz the power, 2015-02-24 07:26:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/931423003/60001

[commit-bot: I haz the power, 2015-02-24 09:24:32]

Committed patchset #7 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=190731