Remove shape-padding

Remove shape-padding

This patch removes shape-padding support, since it can be used only with shape-inside. Shape-inside support has been removed in r170260.

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

[rwlbuis, 2014-03-28 20:57:43]

PTAL

[Zoltan, 2014-03-28 21:18:52]

go clean up go, lgtm!

[rwlbuis, 2014-03-28 21:25:48]

More work related to shape-inside removal. shape-padding only applies to
shape-inside, so it can be removed as well. We decided to do this in a separate
patch since just the shape-inside removal patch was big enough as-is. 

Related WebKit commit:

https://bugs.webkit.org/show_bug.cgi?id=130740

[leviw_travelin_and_unemployed, 2014-03-31 22:25:11]

Wow, yeah, lgtm! Thanks for the cleanup!

[rwlbuis, 2014-03-31 22:26:23]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-03-31 22:26:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rob.buis@samsung.com/216793009/20001

[commit-bot: I haz the power, 2014-03-31 22:27:21]

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

[commit-bot: I haz the power, 2014-03-31 22:27:22]

Failed to apply patch for Source/core/rendering/style/RenderStyle.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/rendering/style/RenderStyle.cpp
  Hunk #1 FAILED at 422.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/core/rendering/style/RenderStyle.cpp.rej

Patch:       Source/core/rendering/style/RenderStyle.cpp
Index: Source/core/rendering/style/RenderStyle.cpp
diff --git a/Source/core/rendering/style/RenderStyle.cpp
b/Source/core/rendering/style/RenderStyle.cpp
index
a4cb88432c2c76f68e13a4f1be202f7770eaf843..7893a889447e9d715301eb61b6116e22769b364e
100644
--- a/Source/core/rendering/style/RenderStyle.cpp
+++ b/Source/core/rendering/style/RenderStyle.cpp
@@ -422,7 +422,6 @@ StyleDifference RenderStyle::visualInvalidationDiff(const
RenderStyle* other, un
             || rareNonInheritedData->m_wrapFlow !=
other->rareNonInheritedData->m_wrapFlow
             || rareNonInheritedData->m_wrapThrough !=
other->rareNonInheritedData->m_wrapThrough
             || rareNonInheritedData->m_shapeMargin !=
other->rareNonInheritedData->m_shapeMargin
-            || rareNonInheritedData->m_shapePadding !=
other->rareNonInheritedData->m_shapePadding
             || rareNonInheritedData->m_order !=
other->rareNonInheritedData->m_order
             || rareNonInheritedData->m_alignContent !=
other->rareNonInheritedData->m_alignContent
             || rareNonInheritedData->m_alignItems !=
other->rareNonInheritedData->m_alignItems

[rwlbuis, 2014-04-01 15:32:26]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-04-01 15:32:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rob.buis@samsung.com/216793009/30001

[commit-bot: I haz the power, 2014-04-01 16:04:51]

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

[commit-bot: I haz the power, 2014-04-01 16:04:52]

Try jobs failed on following builders:
  tryserver.blink on mac_blink_rel

[rwlbuis, 2014-04-01 16:11:41]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-04-01 16:11:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rob.buis@samsung.com/216793009/50001

[commit-bot: I haz the power, 2014-04-01 16:31:12]

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

[commit-bot: I haz the power, 2014-04-01 16:31:13]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[rwlbuis, 2014-04-01 17:09:56]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-04-01 17:10:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rob.buis@samsung.com/216793009/50001

[commit-bot: I haz the power, 2014-04-01 18:18:19]

Change committed as 170590