Avoid unnecessary updateActiveStyle comparing shadow styles.

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc.
  * All rights reserved.
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
@@ skipping 280 matching lines @@
   if (!m_treeBoundaryCrossingRuleSet) {
     m_treeBoundaryCrossingRuleSet = new CSSStyleSheetRuleSubSet();
     treeScope().document().styleEngine().addTreeBoundaryCrossingScope(
         treeScope());
   }
 
   m_treeBoundaryCrossingRuleSet->push_back(
       RuleSubSet::create(parentStyleSheet, sheetIndex, ruleSetForScope));
 }
 
+bool ScopedStyleResolver::hasSameStyles(const ScopedStyleResolver* first,
+                                        const ScopedStyleResolver* second) {
+  unsigned firstCount = first ? first->m_authorStyleSheets.size() : 0;
+  unsigned secondCount = second ? second->m_authorStyleSheets.size() : 0;

[sof, 2017/01/03 18:48:34]

Using size_t would avoid the (unlikely) problem of the two size_t conversions
end up with the same unsigned values, despite being different sizes. And prevent
some compilers from issuing warnings.

[rune, 2017/01/03 21:33:01]

Done.

+  if (firstCount != secondCount)
+    return false;
+  while (firstCount--) {
+    if (first->m_authorStyleSheets[firstCount]->contents() !=
+        second->m_authorStyleSheets[firstCount]->contents())
+      return false;
+  }
+  return true;
+}
+
 DEFINE_TRACE(ScopedStyleResolver::RuleSubSet) {
   visitor->trace(m_parentStyleSheet);
   visitor->trace(m_ruleSet);
 }
 
 }  // namespace blink