Move LengthSizes border-*-radius out of BorderData

third_party/WebKit/Source/core/css/resolver/StyleResolver.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc.
  * All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
@@ skipping 1938 matching lines @@
   new_fill.SetRepeatX(kNoRepeatFill);
   new_fill.SetRepeatY(kNoRepeatFill);
 
   return (old_fill != new_fill || cached_ua_style->background_color !=
                                       state.Style()->BackgroundColor());
 }
 
 bool StyleResolver::HasAuthorBorder(const StyleResolverState& state) {
   const CachedUAStyle* cached_ua_style = state.GetCachedUAStyle();
   return cached_ua_style &&
-         (cached_ua_style->border != state.Style()->Border());
+         (cached_ua_style->border != state.Style()->Border() &&
+          !(cached_ua_style->top_left_ ==

[shend, 2017/05/02 00:01:45]

Hmm, should this be !(top_left == TopLeft && top_right_ == TopRight... etc.? or
just expand it to top_left != TopLeft || top_right_ != TopRight.

[nainar, 2017/05/02 00:42:33]

The != operator doesnt exist could make one but ...

borderData != o.borderData && !(borderRadius1 == o.borderRadius1) &&
!(borderRadius2 == o.borderRadius2) && !(borderRadius3 == o.borderRadius3) &&
!(borderRadius4 == o.borderRadius4)

deMorgans this.. you get

borderData != o.borderData && !(borderRadius1 == o.borderRadius1 ||
borderRadius2 == o.borderRadius2 || borderRadius3 == o.borderRadius3 ||
(borderRadius4 == o.borderRadius4)

right?

[nainar, 2017/05/02 00:49:52]

wait you were right. Done.

+                state.Style()->BorderTopLeftRadius() ||
+            cached_ua_style->top_right_ ==
+                state.Style()->BorderTopRightRadius() ||
+            cached_ua_style->bottom_left_ ==
+                state.Style()->BorderBottomLeftRadius() ||
+            cached_ua_style->bottom_right_ ==
+                state.Style()->BorderBottomRightRadius()));
 }
 
 void StyleResolver::ApplyCallbackSelectors(StyleResolverState& state) {
   RuleSet* watched_selectors_rule_set =
       GetDocument().GetStyleEngine().WatchedSelectorsRuleSet();
   if (!watched_selectors_rule_set)
     return;
 
   ElementRuleCollector collector(state.ElementContext(), selector_filter_,
                                  state.Style());
@@ skipping 47 matching lines @@
 
 DEFINE_TRACE(StyleResolver) {
   visitor->Trace(matched_properties_cache_);
   visitor->Trace(selector_filter_);
   visitor->Trace(style_sharing_lists_);
   visitor->Trace(document_);
   visitor->Trace(tracker_);
 }
 
 }  // namespace blink