Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1129)

Unified Diff: Source/core/paint/ImagePainter.cpp

Issue 594043005: Move paint code from RenderImage/RenderVideo into ImagePainter/VideoPainter. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Addressed comments. Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/paint/ImagePainter.h ('k') | Source/core/paint/VideoPainter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/paint/ImagePainter.cpp
diff --git a/Source/core/paint/ImagePainter.cpp b/Source/core/paint/ImagePainter.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..7759ee0d2d01f4244cb032365570bc9ebedeffb8
--- /dev/null
+++ b/Source/core/paint/ImagePainter.cpp
@@ -0,0 +1,194 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "config.h"
+#include "core/paint/ImagePainter.h"
+
+#include "core/dom/Document.h"
+#include "core/dom/Element.h"
+#include "core/editing/FrameSelection.h"
+#include "core/frame/LocalFrame.h"
+#include "core/html/HTMLAreaElement.h"
+#include "core/html/HTMLImageElement.h"
+#include "core/inspector/InspectorInstrumentation.h"
+#include "core/page/Page.h"
+#include "core/paint/BoxPainter.h"
+#include "core/rendering/PaintInfo.h"
+#include "core/rendering/RenderImage.h"
+#include "core/rendering/RenderReplaced.h"
+#include "core/rendering/TextRunConstructor.h"
+#include "platform/geometry/LayoutPoint.h"
+#include "platform/graphics/GraphicsContextStateSaver.h"
+#include "platform/graphics/Path.h"
+
+namespace blink {
+
+void ImagePainter::paint(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
+{
+ m_renderImage.RenderReplaced::paint(paintInfo, paintOffset);
+
+ if (paintInfo.phase == PaintPhaseOutline)
+ paintAreaElementFocusRing(paintInfo);
+}
+
+void ImagePainter::paintAreaElementFocusRing(PaintInfo& paintInfo)
+{
+ Document& document = m_renderImage.document();
+
+ if (document.printing() || !document.frame()->selection().isFocusedAndActive())
+ return;
+
+ Element* focusedElement = document.focusedElement();
+ if (!isHTMLAreaElement(focusedElement))
+ return;
+
+ HTMLAreaElement& areaElement = toHTMLAreaElement(*focusedElement);
+ if (areaElement.imageElement() != m_renderImage.node())
+ return;
+
+ // Even if the theme handles focus ring drawing for entire elements, it won't do it for
+ // an area within an image, so we don't call RenderTheme::supportsFocusRing here.
+
+ Path path = areaElement.computePath(&m_renderImage);
+ if (path.isEmpty())
+ return;
+
+ RenderStyle* areaElementStyle = areaElement.computedStyle();
+ unsigned short outlineWidth = areaElementStyle->outlineWidth();
+ if (!outlineWidth)
+ return;
+
+ // FIXME: Clip path instead of context when Skia pathops is ready.
+ // https://crbug.com/251206
+ GraphicsContextStateSaver savedContext(*paintInfo.context);
+ paintInfo.context->clip(m_renderImage.absoluteContentBox());
+ paintInfo.context->drawFocusRing(path, outlineWidth,
+ areaElementStyle->outlineOffset(),
+ m_renderImage.resolveColor(areaElementStyle, CSSPropertyOutlineColor));
+}
+
+void ImagePainter::paintReplaced(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
+{
+ LayoutUnit cWidth = m_renderImage.contentWidth();
+ LayoutUnit cHeight = m_renderImage.contentHeight();
+
+ GraphicsContext* context = paintInfo.context;
+
+ if (!m_renderImage.imageResource()->hasImage() || m_renderImage.imageResource()->errorOccurred()) {
+ if (paintInfo.phase == PaintPhaseSelection)
+ return;
+
+ if (cWidth > 2 && cHeight > 2) {
+ const int borderWidth = 1;
+
+ LayoutUnit leftBorder = m_renderImage.borderLeft();
+ LayoutUnit topBorder = m_renderImage.borderTop();
+ LayoutUnit leftPad = m_renderImage.paddingLeft();
+ LayoutUnit topPad = m_renderImage.paddingTop();
+
+ // Draw an outline rect where the image should be.
+ context->setStrokeStyle(SolidStroke);
+ context->setStrokeColor(Color::lightGray);
+ context->setFillColor(Color::transparent);
+ context->drawRect(pixelSnappedIntRect(LayoutRect(paintOffset.x() + leftBorder + leftPad, paintOffset.y() + topBorder + topPad, cWidth, cHeight)));
+
+ bool errorPictureDrawn = false;
+ LayoutSize imageOffset;
+ // When calculating the usable dimensions, exclude the pixels of
+ // the ouline rect so the error image/alt text doesn't draw on it.
+ LayoutUnit usableWidth = cWidth - 2 * borderWidth;
+ LayoutUnit usableHeight = cHeight - 2 * borderWidth;
+
+ RefPtr<Image> image = m_renderImage.imageResource()->image();
+
+ if (m_renderImage.imageResource()->errorOccurred() && !image->isNull() && usableWidth >= image->width() && usableHeight >= image->height()) {
+ float deviceScaleFactor = blink::deviceScaleFactor(m_renderImage.frame());
+ // Call brokenImage() explicitly to ensure we get the broken image icon at the appropriate resolution.
+ pair<Image*, float> brokenImageAndImageScaleFactor = ImageResource::brokenImage(deviceScaleFactor);
+ image = brokenImageAndImageScaleFactor.first;
+ IntSize imageSize = image->size();
+ imageSize.scale(1 / brokenImageAndImageScaleFactor.second);
+ // Center the error image, accounting for border and padding.
+ LayoutUnit centerX = (usableWidth - imageSize.width()) / 2;
+ if (centerX < 0)
+ centerX = 0;
+ LayoutUnit centerY = (usableHeight - imageSize.height()) / 2;
+ if (centerY < 0)
+ centerY = 0;
+ imageOffset = LayoutSize(leftBorder + leftPad + centerX + borderWidth, topBorder + topPad + centerY + borderWidth);
+ context->drawImage(image.get(), pixelSnappedIntRect(LayoutRect(paintOffset + imageOffset, imageSize)), CompositeSourceOver, m_renderImage.shouldRespectImageOrientation());
+ errorPictureDrawn = true;
+ }
+
+ if (!m_renderImage.altText().isEmpty()) {
+ const Font& font = m_renderImage.style()->font();
+ const FontMetrics& fontMetrics = font.fontMetrics();
+ LayoutUnit ascent = fontMetrics.ascent();
+ LayoutPoint textRectOrigin = paintOffset;
+ textRectOrigin.move(leftBorder + leftPad + (RenderImage::paddingWidth / 2) - borderWidth, topBorder + topPad + (RenderImage::paddingHeight / 2) - borderWidth);
+ LayoutPoint textOrigin(textRectOrigin.x(), textRectOrigin.y() + ascent);
+
+ // Only draw the alt text if it'll fit within the content box,
+ // and only if it fits above the error image.
+ TextRun textRun = constructTextRun(&m_renderImage, font, m_renderImage.altText(), m_renderImage.style(), TextRun::AllowTrailingExpansion | TextRun::ForbidLeadingExpansion, DefaultTextRunFlags | RespectDirection);
+ float textWidth = font.width(textRun);
+ TextRunPaintInfo textRunPaintInfo(textRun);
+ textRunPaintInfo.bounds = FloatRect(textRectOrigin, FloatSize(textWidth, fontMetrics.height()));
+ context->setFillColor(m_renderImage.resolveColor(CSSPropertyColor));
+ if (textRun.direction() == RTL) {
+ int availableWidth = cWidth - static_cast<int>(RenderImage::paddingWidth);
+ textOrigin.move(availableWidth - ceilf(textWidth), 0);
+ }
+ if (errorPictureDrawn) {
+ if (usableWidth >= textWidth && fontMetrics.height() <= imageOffset.height())
+ context->drawBidiText(font, textRunPaintInfo, textOrigin);
+ } else if (usableWidth >= textWidth && usableHeight >= fontMetrics.height()) {
+ context->drawBidiText(font, textRunPaintInfo, textOrigin);
+ }
+ }
+ }
+ } else if (m_renderImage.imageResource()->hasImage() && cWidth > 0 && cHeight > 0) {
+ LayoutRect contentRect = m_renderImage.contentBoxRect();
+ contentRect.moveBy(paintOffset);
+ LayoutRect paintRect = m_renderImage.replacedContentRect();
+ paintRect.moveBy(paintOffset);
+ bool clip = !contentRect.contains(paintRect);
+ if (clip) {
+ context->save();
+ context->clip(contentRect);
+ }
+
+ paintIntoRect(context, paintRect);
+
+ if (clip)
+ context->restore();
+ }
+}
+
+void ImagePainter::paintIntoRect(GraphicsContext* context, const LayoutRect& rect)
+{
+ IntRect alignedRect = pixelSnappedIntRect(rect);
+ if (!m_renderImage.imageResource()->hasImage() || m_renderImage.imageResource()->errorOccurred() || alignedRect.width() <= 0 || alignedRect.height() <= 0)
+ return;
+
+ RefPtr<Image> img = m_renderImage.imageResource()->image(alignedRect.width(), alignedRect.height());
+ if (!img || img->isNull())
+ return;
+
+ HTMLImageElement* imageElt = isHTMLImageElement(m_renderImage.node()) ? toHTMLImageElement(m_renderImage.node()) : 0;
+ CompositeOperator compositeOperator = imageElt ? imageElt->compositeOperator() : CompositeSourceOver;
+ Image* image = img.get();
+ InterpolationQuality interpolationQuality = BoxPainter::chooseInterpolationQuality(m_renderImage, context, image, image, alignedRect.size());
+
+ TRACE_EVENT1(TRACE_DISABLED_BY_DEFAULT("devtools.timeline"), "PaintImage", "data", InspectorPaintImageEvent::data(m_renderImage));
+ // FIXME(361045): remove InspectorInstrumentation calls once DevTools Timeline migrates to tracing.
+ InspectorInstrumentation::willPaintImage(&m_renderImage);
+ InterpolationQuality previousInterpolationQuality = context->imageInterpolationQuality();
+ context->setImageInterpolationQuality(interpolationQuality);
+ context->drawImage(image, alignedRect, compositeOperator, m_renderImage.shouldRespectImageOrientation());
+ context->setImageInterpolationQuality(previousInterpolationQuality);
+ InspectorInstrumentation::didPaintImage(&m_renderImage);
+}
+
+} // namespace blink
« no previous file with comments | « Source/core/paint/ImagePainter.h ('k') | Source/core/paint/VideoPainter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698