Avoid RenderObject::paintInvalidationForWholeRenderer() if possible

Avoid RenderObject::paintInvalidationForWholeRenderer() if possible

RenderObject::paintInvalidationForWholeRenderer() violates the principle
of RAL and should be avoided if possible.

- Make the method private and rename to invalidatePaintForWholeRenderer()
- The only remaining caller is RenderObjectChildList::removeChildNode()
- Changed other callers to use setShouldDoFullPaintInvalidation(true)
- In case that shouldDoFullPaintInvalidation(true) is called not during
  FrameView::updateLayoutAndStyleForPainting(), schedule a visual update
  for the page to trigger the next frame if needed.

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

[Xianzhu, 2014-08-14 19:08:52]

Ran layout tests locally and found no over or under repaints. Some duplicated
repaint rects combined. Will update TestExpectations after try bot run.

[dsinclair, 2014-08-14 19:44:40]

lgtm

[robwu, 2014-08-15 22:52:19]

Nitpick: The commit message has a typo (violets -> violates).

[Xianzhu, 2014-08-18 15:31:05]

On 2014/08/15 22:52:19, robwu wrote:
> Nitpick: The commit message has a typo (violets -> violates).

Done :)

[Julien - ping for review, 2014-08-18 16:51:11]

lgtm but we should test some navigation with Chromium before landing (I am
afraid about the potential for regression).

[Xianzhu, 2014-08-18 17:48:08]

On 2014/08/18 16:51:11, Julien Chaffraix - PST wrote:
> lgtm but we should test some navigation with Chromium before landing (I am
> afraid about the potential for regression).

Tried navigating a while and didn't see problems. Perhaps other changes resolved
the issues you saw.

[Xianzhu, 2014-08-18 17:48:13]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2014-08-18 17:50:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/478433002/40001

[commit-bot: I haz the power, 2014-08-18 18:53:08]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/20246)

[commit-bot: I haz the power, 2014-08-18 19:18:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-18 19:18:58]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)

[Xianzhu, 2014-08-18 20:26:01]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2014-08-18 20:27:06]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/478433002/60001

[commit-bot: I haz the power, 2014-08-18 21:35:14]

Committed patchset #4 (60001) as 180496