From 9f5b097879c44635b59d31e14a8fe8c2ba6a06ef Mon Sep 17 00:00:00 2001
From: Andreas Svensson <andreas@syranide.com>
Date: Tue, 29 Jul 2014 23:06:04 +0200
Subject: [PATCH] Apply CSS shorthand expansion to IE8 only

---
 src/browser/ui/dom/CSSPropertyOperations.js | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/browser/ui/dom/CSSPropertyOperations.js b/src/browser/ui/dom/CSSPropertyOperations.js
index 46cb6109a266..37d0f96e3cae 100644
--- a/src/browser/ui/dom/CSSPropertyOperations.js
+++ b/src/browser/ui/dom/CSSPropertyOperations.js
@@ -32,8 +32,15 @@ var processStyleName = memoizeStringOnly(function(styleName) {
   return hyphenateStyleName(styleName);
 });
 
+var hasShorthandPropertyBug = false;
 var styleFloatAccessor = 'cssFloat';
 if (ExecutionEnvironment.canUseDOM) {
+  try {
+    // IE8 throws "Invalid argument." if resetting shorthand style properties.
+    document.createElement('div').style.font = '';
+  } catch (e) {
+    hasShorthandPropertyBug = true;
+  }
   // IE8 only supports accessing cssFloat (standard) as styleFloat
   if (document.documentElement.style.cssFloat === undefined) {
     styleFloatAccessor = 'styleFloat';
@@ -118,7 +125,7 @@ var CSSPropertyOperations = {
       }
       if (styleValue) {
         style[styleName] = styleValue;
-      } else {
+      } else if (hasShorthandPropertyBug) {
         var expansion = CSSProperty.shorthandPropertyExpansions[styleName];
         if (expansion) {
           // Shorthand property that IE8 won't like unsetting, so unset each
@@ -129,6 +136,8 @@ var CSSPropertyOperations = {
         } else {
           style[styleName] = '';
         }
+      } else {
+        style[styleName] = '';
       }
     }
   }