Revert of Moved parent_->SchedulePaint() to Views::SetFillsBoundsOpaquely() and Views::ScheduleP

Revert of Moved parent_->SchedulePaint() to Views::SetFillsBoundsOpaquely() and Views::SchedulePaint(). (patchset #8 id:140001 of https://codereview.chromium.org/1461673002/ )

Reason for revert:
These tests leak

https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/8315

Original issue's description:
> This CL fixes a more general issue in Views that had a quick fix
> that landed here: https://codereview.chromium.org/1460733002/
>
> The general problem is that the layer owned by a parent View can
> become dirty/stale when a child view is painting to it's own layer.
> This CL fixes the problem by scheduling paints on the parent view
> when it may be dirty.
>
> BUG=551492, 546683
> TEST=ViewLayerTest.NoCrashWhenParentlessViewSchedulesPaintOnParent
>      ViewLayerTest.ScheduledRectsInParentAfterSchedulingPaint
>      ViewLayerTest.ParentPaintWhenSwitchingPaintToLayerFromFalseToTrue
>      ViewLayerTest.NoParentPaintWhenSwitchingPaintToLayerFromTrueToTrue
>
> Committed: https://crrev.com/c8761106916aac604f30fb2223cdbfb2bd3ffda3
> Cr-Commit-Position: refs/heads/master@{#367536}

TBR=sadrul@chromium.org,jonross@chromium.org,sky@chromium.org,bruthig@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=551492, 546683
Committed: https://crrev.com/70e4baf6b13f7e4fda458ea4c9f684c55bed957c
Cr-Commit-Position: refs/heads/master@{#367563}

[Ken Rockot(use gerrit already), 2016-01-05 17:41:43]

Created Revert of Moved parent_->SchedulePaint() to
Views::SetFillsBoundsOpaquely() and Views::SchedulePaint().

[commit-bot: I haz the power, 2016-01-05 17:45:31]

Description was changed from

==========
Revert of Moved parent_->SchedulePaint() to Views::SetFillsBoundsOpaquely() and
Views::SchedulePaint(). (patchset #8 id:140001 of
https://codereview.chromium.org/1461673002/ )

Reason for revert:
These tests leak

https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

Original issue's description:
> This CL fixes a more general issue in Views that had a quick fix
> that landed here: https://codereview.chromium.org/1460733002/
> 
> The general problem is that the layer owned by a parent View can
> become dirty/stale when a child view is painting to it's own layer.
> This CL fixes the problem by scheduling paints on the parent view
> when it may be dirty.
> 
> BUG=551492, 546683
> TEST=ViewLayerTest.NoCrashWhenParentlessViewSchedulesPaintOnParent
>      ViewLayerTest.ScheduledRectsInParentAfterSchedulingPaint
>      ViewLayerTest.ParentPaintWhenSwitchingPaintToLayerFromFalseToTrue
>      ViewLayerTest.NoParentPaintWhenSwitchingPaintToLayerFromTrueToTrue
> 
> Committed: https://crrev.com/c8761106916aac604f30fb2223cdbfb2bd3ffda3
> Cr-Commit-Position: refs/heads/master@{#367536}

TBR=sadrul@chromium.org,jonross@chromium.org,sky@chromium.org,bruthig@chromiu...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=551492, 546683
==========

to

==========
Revert of Moved parent_->SchedulePaint() to Views::SetFillsBoundsOpaquely() and
Views::SchedulePaint(). (patchset #8 id:140001 of
https://codereview.chromium.org/1461673002/ )

Reason for revert:
These tests leak

https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

Original issue's description:
> This CL fixes a more general issue in Views that had a quick fix
> that landed here: https://codereview.chromium.org/1460733002/
>
> The general problem is that the layer owned by a parent View can
> become dirty/stale when a child view is painting to it's own layer.
> This CL fixes the problem by scheduling paints on the parent view
> when it may be dirty.
>
> BUG=551492, 546683
> TEST=ViewLayerTest.NoCrashWhenParentlessViewSchedulesPaintOnParent
>      ViewLayerTest.ScheduledRectsInParentAfterSchedulingPaint
>      ViewLayerTest.ParentPaintWhenSwitchingPaintToLayerFromFalseToTrue
>      ViewLayerTest.NoParentPaintWhenSwitchingPaintToLayerFromTrueToTrue
>
> Committed: https://crrev.com/c8761106916aac604f30fb2223cdbfb2bd3ffda3
> Cr-Commit-Position: refs/heads/master@{#367536}

TBR=sadrul@chromium.org,jonross@chromium.org,sky@chromium.org,bruthig@chromiu...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=551492, 546683

Committed: https://crrev.com/70e4baf6b13f7e4fda458ea4c9f684c55bed957c
Cr-Commit-Position: refs/heads/master@{#367563}
==========

[Dan Beam, 2016-01-05 18:06:08]

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

[Dan Beam, 2016-01-05 18:06:08]

https://codereview.chromium.org/1560723002/diff/1/ui/views/view_unittest.cc
File ui/views/view_unittest.cc (left):

https://codereview.chromium.org/1560723002/diff/1/ui/views/view_unittest.cc#o...
ui/views/view_unittest.cc:3992: TestView* v = new TestView;
TestView v;
SchedulePaintOnParent(&v);

would probably fix

[bruthig, 2016-01-06 16:37:05]

Re-landed original patch set here: https://codereview.chromium.org/1566693002/