Delete PaintInvalidationState. It's unused.

Delete PaintInvalidationState. It's unused.


Delete selection paint invalidation code.

There is a slight change in behavior in FrameSelection::revealSelection.
If you have a non-collapsed selection, then we'll center the start
of the selection instead of the whole selection in some cases. There's
a ton of callers of this code, so it's hard to be sure if any of this
actually changes behavior for sky. In manual testing, I couldn't find
any scenarios where there was a difference. Almost universally,
when we call revealSelection, we have a CaretSelection. The only
case I could think of where we have a RangeSelection is when
modifying an off-screen selection (e.g. shift+right), but in that case
we pass the RevealExtent option, so this patch doesn't change behavior
there.

Removing that caller makes all the rest of this rect computing
code into dead code.

R=abarth@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/cbd33628df59e51aac535c2c5daa8fd5f0eedd33

[ojan, 2015-01-14 21:18:42]

This depends on https://codereview.chromium.org/847303003

[abarth-chromium, 2015-01-14 21:20:44]

abarth@chromium.org changed reviewers:
+ abarth@chromium.org

[abarth-chromium, 2015-01-14 21:20:45]

lgtm

[ojan, 2015-01-14 21:25:53]

Committed patchset #2 (id:20001) manually as
cbd33628df59e51aac535c2c5daa8fd5f0eedd33 (presubmit successful).