Rename Repaint to Paint Invalidation Part 2

Rename Repaint to Paint Invalidation Part 2

This CL refactors the FrameView methods to be paintInvalidation instead
of repaint.

I have left the repaint rect output to be 'repaint' for now as that
seems like a large and scary rebaseline when it lands.

I also did not change the slowRepaints methods as they were previously
mentioned as not having to do with paint invalidation.

BUG=338691
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175492

[dsinclair, 2014-06-02 19:17:14]

PTAL.

[Julien - ping for review, 2014-06-03 01:00:20]

lgtm but the description is too wide (it should be <80 characters sir!)

https://codereview.chromium.org/306413002/diff/20001/Source/core/frame/FrameV...
File Source/core/frame/FrameView.cpp (right):

https://codereview.chromium.org/306413002/diff/20001/Source/core/frame/FrameV...
Source/core/frame/FrameView.cpp:359: IntRect paintInavlidationRect = rect;
typo: paintIn*va*lidationRect

https://codereview.chromium.org/306413002/diff/20001/Source/core/frame/FrameV...
Source/core/frame/FrameView.cpp:1761: ASSERT(paintInvalidationAllowed());
I didn't catch that earlier but this should have a verb as we are renaming it:
paintInvalidationIsAllowed()

https://codereview.chromium.org/306413002/diff/20001/Source/platform/scroll/S...
File Source/platform/scroll/ScrollView.cpp (right):

https://codereview.chromium.org/306413002/diff/20001/Source/platform/scroll/S...
Source/platform/scroll/ScrollView.cpp:769: void
ScrollView::invalidatedContentRectangleForPaint(const IntRect& rect)
contentRectangleForPaintInvalidation?

[dsinclair, 2014-06-03 14:06:17]

https://codereview.chromium.org/306413002/diff/20001/Source/core/frame/FrameV...
File Source/core/frame/FrameView.cpp (right):

https://codereview.chromium.org/306413002/diff/20001/Source/core/frame/FrameV...
Source/core/frame/FrameView.cpp:359: IntRect paintInavlidationRect = rect;
On 2014/06/03 01:00:20, Julien Chaffraix - PST wrote:
> typo: paintIn*va*lidationRect

Done.

https://codereview.chromium.org/306413002/diff/20001/Source/core/frame/FrameV...
Source/core/frame/FrameView.cpp:1761: ASSERT(paintInvalidationAllowed());
On 2014/06/03 01:00:20, Julien Chaffraix - PST wrote:
> I didn't catch that earlier but this should have a verb as we are renaming it:
> paintInvalidationIsAllowed()

Done.

https://codereview.chromium.org/306413002/diff/20001/Source/platform/scroll/S...
File Source/platform/scroll/ScrollView.cpp (right):

https://codereview.chromium.org/306413002/diff/20001/Source/platform/scroll/S...
Source/platform/scroll/ScrollView.cpp:769: void
ScrollView::invalidatedContentRectangleForPaint(const IntRect& rect)
On 2014/06/03 01:00:20, Julien Chaffraix - PST wrote:
> contentRectangleForPaintInvalidation?

Done.

[dsinclair, 2014-06-04 13:30:06]

ping ojan, esprehn for Source/web/OWNERS.

[eseidel, 2014-06-04 14:50:53]

rslgtm

[dsinclair, 2014-06-04 14:58:58]

The CQ bit was checked by dsinclair@chromium.org

[commit-bot: I haz the power, 2014-06-04 14:59:39]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dsinclair@chromium.org/306413002/50001

[commit-bot: I haz the power, 2014-06-04 15:20:03]

Change committed as 175492