Commit 9924abf1 authored by danakj@chromium.org's avatar danakj@chromium.org

Don't convert invalidations to a FloatRect.

We just turn it back into an IntRect on the cc side.

Depends on: https://codereview.chromium.org/647253002/
Depends on: https://codereview.chromium.org/655863002/

R=enne
BUG=342848

Review URL: https://codereview.chromium.org/646133002

git-svn-id: svn://svn.chromium.org/blink/trunk@183693 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent af1dc8d8
...@@ -567,7 +567,7 @@ void Canvas2DLayerBridge::finalizeFrame(const FloatRect &dirtyRect) ...@@ -567,7 +567,7 @@ void Canvas2DLayerBridge::finalizeFrame(const FloatRect &dirtyRect)
{ {
ASSERT(!m_destructionInProgress); ASSERT(!m_destructionInProgress);
Canvas2DLayerManager::get().layerDidDraw(this); Canvas2DLayerManager::get().layerDidDraw(this);
m_layer->layer()->invalidateRect(dirtyRect); m_layer->layer()->invalidateRect(enclosingIntRect(dirtyRect));
m_didRecordDrawCommand = true; m_didRecordDrawCommand = true;
} }
......
...@@ -898,7 +898,7 @@ void GraphicsLayer::setNeedsDisplay() ...@@ -898,7 +898,7 @@ void GraphicsLayer::setNeedsDisplay()
} }
} }
void GraphicsLayer::setNeedsDisplayInRect(const FloatRect& rect, PaintInvalidationReason invalidationReason) void GraphicsLayer::setNeedsDisplayInRect(const IntRect& rect, PaintInvalidationReason invalidationReason)
{ {
if (drawsContent()) { if (drawsContent()) {
m_layer->layer()->invalidateRect(rect); m_layer->layer()->invalidateRect(rect);
......
...@@ -182,8 +182,8 @@ public: ...@@ -182,8 +182,8 @@ public:
void setPaintingPhase(GraphicsLayerPaintingPhase); void setPaintingPhase(GraphicsLayerPaintingPhase);
void setNeedsDisplay(); void setNeedsDisplay();
// mark the given rect (in layer coords) as needing dispay. Never goes deep. // Mark the given rect (in layer coords) as needing display. Never goes deep.
void setNeedsDisplayInRect(const FloatRect&, PaintInvalidationReason); void setNeedsDisplayInRect(const IntRect&, PaintInvalidationReason);
void setContentsNeedsDisplay(); void setContentsNeedsDisplay();
......
...@@ -372,7 +372,7 @@ void WebPopupMenuImpl::invalidateContentsAndRootView(const IntRect& paintRect) ...@@ -372,7 +372,7 @@ void WebPopupMenuImpl::invalidateContentsAndRootView(const IntRect& paintRect)
if (m_client) if (m_client)
m_client->didInvalidateRect(paintRect); m_client->didInvalidateRect(paintRect);
if (m_rootLayer) if (m_rootLayer)
m_rootLayer->layer()->invalidateRect(FloatRect(paintRect)); m_rootLayer->layer()->invalidateRect(paintRect);
} }
void WebPopupMenuImpl::invalidateContentsForSlowScroll(const IntRect& updateRect) void WebPopupMenuImpl::invalidateContentsForSlowScroll(const IntRect& updateRect)
......
...@@ -47,7 +47,6 @@ class WebFilterOperations; ...@@ -47,7 +47,6 @@ class WebFilterOperations;
class WebLayerClient; class WebLayerClient;
class WebLayerScrollClient; class WebLayerScrollClient;
struct WebFloatPoint; struct WebFloatPoint;
struct WebFloatRect;
struct WebLayerPositionConstraint; struct WebLayerPositionConstraint;
class WebLayer { class WebLayer {
...@@ -58,7 +57,7 @@ public: ...@@ -58,7 +57,7 @@ public:
virtual int id() const = 0; virtual int id() const = 0;
// Sets a region of the layer as invalid, i.e. needs to update its content. // Sets a region of the layer as invalid, i.e. needs to update its content.
virtual void invalidateRect(const WebFloatRect&) = 0; virtual void invalidateRect(const WebRect&) = 0;
// Sets the entire layer as invalid, i.e. needs to update its content. // Sets the entire layer as invalid, i.e. needs to update its content.
virtual void invalidate() = 0; virtual void invalidate() = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment