Title: [174018] trunk/Source/WebCore
- Revision
- 174018
- Author
- [email protected]
- Date
- 2014-09-26 14:05:01 -0700 (Fri, 26 Sep 2014)
Log Message
REGRESSION (r173988): Fix unused variable warning in PDFDocumentImage.cpp
Fixes the following build failure in release builds:
WebCore/platform/graphics/cg/PDFDocumentImage.cpp:230:12: error: unused variable 'pageCount' [-Werror,-Wunused-variable]
* platform/graphics/cg/PDFDocumentImage.cpp:
(WebCore::PDFDocumentImage::computeBoundsForCurrentPage): Change
ASSERT() to call pageCount().
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (174017 => 174018)
--- trunk/Source/WebCore/ChangeLog 2014-09-26 19:02:42 UTC (rev 174017)
+++ trunk/Source/WebCore/ChangeLog 2014-09-26 21:05:01 UTC (rev 174018)
@@ -1,3 +1,15 @@
+2014-09-26 David Kilzer <[email protected]>
+
+ REGRESSION (r173988): Fix unused variable warning in PDFDocumentImage.cpp
+
+ Fixes the following build failure in release builds:
+
+ WebCore/platform/graphics/cg/PDFDocumentImage.cpp:230:12: error: unused variable 'pageCount' [-Werror,-Wunused-variable]
+
+ * platform/graphics/cg/PDFDocumentImage.cpp:
+ (WebCore::PDFDocumentImage::computeBoundsForCurrentPage): Change
+ ASSERT() to call pageCount().
+
2014-09-26 Brian J. Burg <[email protected]>
StorageTracker::deleteOrigin being called off the main thread (ASSERTs in inspector/test-harness-trivially-works.html test)
Modified: trunk/Source/WebCore/platform/graphics/cg/PDFDocumentImage.cpp (174017 => 174018)
--- trunk/Source/WebCore/platform/graphics/cg/PDFDocumentImage.cpp 2014-09-26 19:02:42 UTC (rev 174017)
+++ trunk/Source/WebCore/platform/graphics/cg/PDFDocumentImage.cpp 2014-09-26 21:05:01 UTC (rev 174018)
@@ -227,8 +227,7 @@
void PDFDocumentImage::computeBoundsForCurrentPage()
{
- size_t pageCount = CGPDFDocumentGetNumberOfPages(m_document.get());
- ASSERT(pageCount > 0);
+ ASSERT(pageCount() > 0);
CGPDFPageRef cgPage = CGPDFDocumentGetPage(m_document.get(), 1);
CGRect mediaBox = CGPDFPageGetBoxRect(cgPage, kCGPDFMediaBox);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
