Title: [202440] trunk/Source/WebCore
Revision
202440
Author
[email protected]
Date
2016-06-24 13:14:31 -0700 (Fri, 24 Jun 2016)

Log Message

Remove unused and static return value from InspectorStyle::populateAllProperties
https://bugs.webkit.org/show_bug.cgi?id=159069

Patch by Joseph Pecoraro <[email protected]> on 2016-06-24
Reviewed by Andreas Kling.

* inspector/InspectorStyleSheet.cpp:
(WebCore::InspectorStyle::populateAllProperties):
* inspector/InspectorStyleSheet.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (202439 => 202440)


--- trunk/Source/WebCore/ChangeLog	2016-06-24 19:40:19 UTC (rev 202439)
+++ trunk/Source/WebCore/ChangeLog	2016-06-24 20:14:31 UTC (rev 202440)
@@ -1,3 +1,14 @@
+2016-06-24  Joseph Pecoraro  <[email protected]>
+
+        Remove unused and static return value from InspectorStyle::populateAllProperties
+        https://bugs.webkit.org/show_bug.cgi?id=159069
+
+        Reviewed by Andreas Kling.
+
+        * inspector/InspectorStyleSheet.cpp:
+        (WebCore::InspectorStyle::populateAllProperties):
+        * inspector/InspectorStyleSheet.h:
+
 2016-06-21  Anders Carlsson  <[email protected]>
 
         Rename NoncopyableFunction to Function

Modified: trunk/Source/WebCore/inspector/InspectorStyleSheet.cpp (202439 => 202440)


--- trunk/Source/WebCore/inspector/InspectorStyleSheet.cpp	2016-06-24 19:40:19 UTC (rev 202439)
+++ trunk/Source/WebCore/inspector/InspectorStyleSheet.cpp	2016-06-24 20:14:31 UTC (rev 202440)
@@ -357,7 +357,7 @@
     return name.convertToASCIILowercase();
 }
 
-bool InspectorStyle::populateAllProperties(Vector<InspectorStyleProperty>* result) const
+void InspectorStyle::populateAllProperties(Vector<InspectorStyleProperty>* result) const
 {
     HashSet<String> sourcePropertyNames;
 
@@ -380,8 +380,6 @@
         if (sourcePropertyNames.add(lowercasePropertyName(name)))
             result->append(InspectorStyleProperty(CSSPropertySourceData(name, m_style->getPropertyValue(name), !m_style->getPropertyPriority(name).isEmpty(), true, SourceRange()), false, false));
     }
-
-    return true;
 }
 
 Ref<Inspector::Protocol::CSS::CSSStyle> InspectorStyle::styleWithProperties() const

Modified: trunk/Source/WebCore/inspector/InspectorStyleSheet.h (202439 => 202440)


--- trunk/Source/WebCore/inspector/InspectorStyleSheet.h	2016-06-24 19:40:19 UTC (rev 202439)
+++ trunk/Source/WebCore/inspector/InspectorStyleSheet.h	2016-06-24 20:14:31 UTC (rev 202440)
@@ -140,7 +140,7 @@
 private:
     InspectorStyle(const InspectorCSSId& styleId, RefPtr<CSSStyleDeclaration>&&, InspectorStyleSheet* parentStyleSheet);
 
-    bool populateAllProperties(Vector<InspectorStyleProperty>* result) const;
+    void populateAllProperties(Vector<InspectorStyleProperty>* result) const;
     Ref<Inspector::Protocol::CSS::CSSStyle> styleWithProperties() const;
     RefPtr<CSSRuleSourceData> extractSourceData() const;
     String shorthandValue(const String& shorthandProperty) const;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to