Title: [151863] trunk/Source/WebCore
- Revision
- 151863
- Author
- [email protected]
- Date
- 2013-06-21 15:50:01 -0700 (Fri, 21 Jun 2013)
Log Message
[MediaStream]: Remove ``>= 0'' assertion from a size_t variable
https://bugs.webkit.org/show_bug.cgi?id=117890
Patch by Sergio Correia <[email protected]> on 2013-06-21
Reviewed by Adam Barth.
This patch fixes a strict build by removing the ``>= 0'' assertion from
a size_t (unsigned type) variable in RTCStatsResponse.cpp.
No new tests; build fix.
* Modules/mediastream/RTCStatsResponse.cpp:
(WebCore::RTCStatsResponse::addStatistic): Remove ``>=0''assertion,
since the variable is unsigned and is always >= 0.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (151862 => 151863)
--- trunk/Source/WebCore/ChangeLog 2013-06-21 22:20:04 UTC (rev 151862)
+++ trunk/Source/WebCore/ChangeLog 2013-06-21 22:50:01 UTC (rev 151863)
@@ -1,3 +1,19 @@
+2013-06-21 Sergio Correia <[email protected]>
+
+ [MediaStream]: Remove ``>= 0'' assertion from a size_t variable
+ https://bugs.webkit.org/show_bug.cgi?id=117890
+
+ Reviewed by Adam Barth.
+
+ This patch fixes a strict build by removing the ``>= 0'' assertion from
+ a size_t (unsigned type) variable in RTCStatsResponse.cpp.
+
+ No new tests; build fix.
+
+ * Modules/mediastream/RTCStatsResponse.cpp:
+ (WebCore::RTCStatsResponse::addStatistic): Remove ``>=0''assertion,
+ since the variable is unsigned and is always >= 0.
+
2013-06-21 Alex Christensen <[email protected]>
GraphicsContext3DNEON.h cannot be found on Windows and does not need to be found.
Modified: trunk/Source/WebCore/Modules/mediastream/RTCStatsResponse.cpp (151862 => 151863)
--- trunk/Source/WebCore/Modules/mediastream/RTCStatsResponse.cpp 2013-06-21 22:20:04 UTC (rev 151862)
+++ trunk/Source/WebCore/Modules/mediastream/RTCStatsResponse.cpp 2013-06-21 22:50:01 UTC (rev 151863)
@@ -55,7 +55,7 @@
void RTCStatsResponse::addStatistic(size_t report, String name, String value)
{
- ASSERT_WITH_SECURITY_IMPLICATION(report >= 0 && report < m_result.size());
+ ASSERT_WITH_SECURITY_IMPLICATION(report < m_result.size());
m_result[report]->addStatistic(name, value);
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
