Changeset 112492 in webkit
- Timestamp:
- Mar 28, 2012 7:11:40 PM (12 years ago)
- Location:
- trunk/Source/WebCore
- Files:
-
- 2 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r112490 r112492 1 2012-03-28 Emil A Eklund <eae@chromium.org> 2 3 Fix rounding in RenderInline::paintOutlineForLine 4 https://bugs.webkit.org/show_bug.cgi?id=82540 5 6 Reviewed by Eric Seidel. 7 8 No new tests, no change in functionality. 9 10 * rendering/RenderInline.cpp: 11 (WebCore::RenderInline::paintOutlineForLine): 12 Change paintOutlineForLine to pixel snap the left and right edges of the 13 lines instead of adding the offset to an already rounded number. 14 1 15 2012-03-28 David Grogan <dgrogan@chromium.org> 2 16 -
trunk/Source/WebCore/rendering/RenderInline.cpp
r112475 r112492 1466 1466 1467 1467 IntRect pixelSnappedBox = pixelSnappedIntRect(box); 1468 IntRect pixelSnappedLastLine = pixelSnappedIntRect(paintOffset.x() + lastline.x(), 0, lastline.width(), 0); 1469 IntRect pixelSnappedNextLine = pixelSnappedIntRect(paintOffset.x() + nextline.x(), 0, nextline.width(), 0); 1468 1470 1469 1471 // left edge … … 1495 1497 pixelSnappedBox.x() - outlineWidth, 1496 1498 pixelSnappedBox.y() - outlineWidth, 1497 min(pixelSnappedBox.maxX() + outlineWidth, (lastline.isEmpty() ? 1000000 : p aintOffset.x() + lastline.x())),1499 min(pixelSnappedBox.maxX() + outlineWidth, (lastline.isEmpty() ? 1000000 : pixelSnappedLastLine.x())), 1498 1500 pixelSnappedBox.y(), 1499 1501 BSTop, outlineColor, outlineStyle, … … 1504 1506 if (lastline.maxX() < thisline.maxX()) 1505 1507 drawLineForBoxSide(graphicsContext, 1506 max(lastline.isEmpty() ? -1000000 : p aintOffset.x() + lastline.maxX(), pixelSnappedBox.x() - outlineWidth),1508 max(lastline.isEmpty() ? -1000000 : pixelSnappedLastLine.maxX(), pixelSnappedBox.x() - outlineWidth), 1507 1509 pixelSnappedBox.y() - outlineWidth, 1508 1510 pixelSnappedBox.maxX() + outlineWidth, … … 1528 1530 pixelSnappedBox.x() - outlineWidth, 1529 1531 pixelSnappedBox.maxY(), 1530 min(pixelSnappedBox.maxX() + outlineWidth, !nextline.isEmpty() ? p aintOffset.x() + nextline.x() + 1 : 1000000),1532 min(pixelSnappedBox.maxX() + outlineWidth, !nextline.isEmpty() ? pixelSnappedNextLine.x() + 1 : 1000000), 1531 1533 pixelSnappedBox.maxY() + outlineWidth, 1532 1534 BSBottom, outlineColor, outlineStyle, … … 1537 1539 if (nextline.maxX() < thisline.maxX()) 1538 1540 drawLineForBoxSide(graphicsContext, 1539 max(!nextline.isEmpty() ? p aintOffset.x() + nextline.maxX() : -1000000, pixelSnappedBox.x() - outlineWidth),1541 max(!nextline.isEmpty() ? pixelSnappedNextLine.maxX() : -1000000, pixelSnappedBox.x() - outlineWidth), 1540 1542 pixelSnappedBox.maxY(), 1541 1543 pixelSnappedBox.maxX() + outlineWidth,
Note: See TracChangeset
for help on using the changeset viewer.