Issue #1673 - Part 2: Make tab-size animatable and fix typos.

There were a few typos in the previous patch and this patch also makes tab-size animatable which didn't really require much of a change at all.

dom/animation/test/mozilla/file_discrete-animations.html

@@ -75,11 +75,6 @@ const gMozillaSpecificProperties = {
     from: "ignore",
     to: "stretch-to-fit"
   },
-  "-moz-tab-size": {
-    // https://drafts.csswg.org/css-text-3/#propdef-tab-size
-    from: "1",
-    to: "5"
-  },
   "-moz-text-size-adjust": {
     // https://drafts.csswg.org/css-size-adjust/#propdef-text-size-adjust
     from: "none",

layout/generic/nsTextFrame.cpp

@@ -3435,7 +3435,7 @@ PropertyProvider::GetSpacingInternal(Range aRange, Spacing* aSpacing,
   if (!aIgnoreTabs) {
     gfxFloat tabWidth = ComputeTabWidthAppUnits(mFrame, mTextRun);
     if (tabWidth > 0) {
-      CalcTabWidths(aRange);
+      CalcTabWidths(aRange, tabWidth);
       if (mTabWidths) {
         mTabWidths->ApplySpacing(aSpacing,
                                  aRange.start - mStart.GetSkippedOffset(),
@@ -3466,13 +3466,13 @@ PropertyProvider::GetSpacingInternal(Range aRange, Spacing* aSpacing,
 // aX and the result are in whole appunits.
 static gfxFloat
 AdvanceToNextTab(gfxFloat aX, nsIFrame* aFrame,
-                 gfxTextRun* aTextRun, gfxFloat* aTabWidth)
+                 gfxTextRun* aTextRun, gfxFloat aTabWidth)
 {
 
   // Advance aX to the next multiple of *aCachedTabWidth. We must advance
   // by at least 1 appunit.
   // XXX should we make this 1 CSS pixel?
-  return ceil((aX + 1)/ aTabWidth) * aTabWidth;
+  return NS_round((aX + 1) / aTabWidth) * aTabWidth;
 }
 
 void

layout/style/nsCSSPropList.h

@@ -3888,7 +3888,7 @@ CSS_PROP_TEXT(
     VARIANT_INHERIT | VARIANT_LNCALC,
     nullptr,
     offsetof(nsStyleText, mTabSize),
-    eStyleAnimType_Discrete)
+    eStyleAnimType_Coord)
 CSS_PROP_TABLE(
     table-layout,
     table_layout,

layout/style/nsRuleNode.cpp

@@ -4509,7 +4509,7 @@ TruncateStringToSingleGrapheme(nsAString& aStr)
 
 struct LengthNumberCalcObj
 {
-  float LengthNumberCalcObj;
+  float mValue;
   bool mIsNumber;
 };
 

layout/style/test/test_transitions_per_property.html

@@ -248,6 +248,8 @@ var supported_properties = {
     // test_length_percent_calc_transition.
     "stroke-width": [ test_length_transition_svg, test_percent_transition,
                       test_length_clamped_svg, test_percent_clamped ],
+    "-moz-tab-size": [ test_float_zeroToOne_transition,
+                       test_float_aboveOne_transition, test_length_clamped ],
     "text-decoration": [ test_color_shorthand_transition,
                          test_true_currentcolor_shorthand_transition ],
     "text-decoration-color": [ test_color_transition,