Revert of Refactor ContentViewClient (4/6)

Revert of Refactor ContentViewClient (4/6) (patchset #4 id:140001 of https://codereview.chromium.org/2682593002/ )

Reason for revert:
Causing major alignment issues for custom tabs

BUG=691718
Original issue's description:
> Refactor ContentViewClient (4/6)
> 
> Moved following interface methods to ViewAndroid:
>  - onBackgroundColorChanged
>  - onTopControlsChanged
>  - onBottomControlsChanged
>  - onStartContentIntent
> 
> Remove following as they are not used:
>  - getSystemWindowInsetTop
>  - getSystemWindowInsetLeft
>  - getSystemWindowInsetRight
> 
> BUG=620172
> 
> Review-Url: https://codereview.chromium.org/2682593002
> Cr-Commit-Position: refs/heads/master@{#449186}
> Committed: https://chromium.googlesource.com/chromium/src/+/d95ac3f4aad80e2102fd8cf957dc2e265c0f02f7

TBR=boliu@chromium.org,dtrainor@chromium.org,jinsukkim@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=620172
Review-Url: https://codereview.chromium.org/2694273002
Cr-Commit-Position: refs/heads/master@{#450616}
Committed: https://chromium.googlesource.com/chromium/src/+/cefe29f6134013119b32e9e96ffa8ad3f74c5304

[amineer, 2017-02-14 18:27:44]

The CQ bit was checked by amineer@chromium.org

[amineer, 2017-02-14 18:27:45]

Created Revert of Refactor ContentViewClient (4/6)

[commit-bot: I haz the power, 2017-02-14 18:28:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[amineer, 2017-02-14 18:43:02]

The CQ bit was unchecked by amineer@chromium.org

[amineer, 2017-02-14 18:43:53]

On 2017/02/14 18:28:44, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

Unchecking the "commit" on this patch as the regression is not as bad as
initially feared.  Given the size of the change, and the fact that this is #4 of
6 in a series, perhaps fixing would be the best course of action rather than
attempting a revert.  Jinsuk, please advise.

[boliu, 2017-02-14 18:47:17]

On 2017/02/14 18:43:53, amineer wrote:
> On 2017/02/14 18:28:44, commit-bot: I haz the power wrote:
> > CQ is trying da patch. Follow status at
> >  
> >
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
> 
> Unchecking the "commit" on this patch as the regression is not as bad as
> initially feared.  Given the size of the change, and the fact that this is #4
of
> 6 in a series, perhaps fixing would be the best course of action rather than
> attempting a revert.  Jinsuk, please advise.

just revert imo, shouldn't be very dangerous, and cq will catch and build
problems

[David Trainor- moved to gerrit, 2017-02-14 19:30:40]

rubber stamp lgtm for revert.

[amineer, 2017-02-15 00:08:50]

The CQ bit was checked by amineer@chromium.org

[commit-bot: I haz the power, 2017-02-15 00:09:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-15 07:00:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487117330666980, "parent_rev":
"53d28e3d9446105d1609fba58575bc44b50127b8", "commit_rev":
"cefe29f6134013119b32e9e96ffa8ad3f74c5304"}

[commit-bot: I haz the power, 2017-02-15 07:01:29]

Description was changed from

==========
Revert of Refactor ContentViewClient (4/6) (patchset #4 id:140001 of
https://codereview.chromium.org/2682593002/ )

Reason for revert:
Causing major alignment issues for custom tabs

BUG=691718

Original issue's description:
> Refactor ContentViewClient (4/6)
> 
> Moved following interface methods to ViewAndroid:
>  - onBackgroundColorChanged
>  - onTopControlsChanged
>  - onBottomControlsChanged
>  - onStartContentIntent
> 
> Remove following as they are not used:
>  - getSystemWindowInsetTop
>  - getSystemWindowInsetLeft
>  - getSystemWindowInsetRight
> 
> BUG=620172
> 
> Review-Url: https://codereview.chromium.org/2682593002
> Cr-Commit-Position: refs/heads/master@{#449186}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d95ac3f4aad80e2102fd8cf957dc...

TBR=boliu@chromium.org,dtrainor@chromium.org,jinsukkim@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=620172
==========

to

==========
Revert of Refactor ContentViewClient (4/6) (patchset #4 id:140001 of
https://codereview.chromium.org/2682593002/ )

Reason for revert:
Causing major alignment issues for custom tabs

BUG=691718

Original issue's description:
> Refactor ContentViewClient (4/6)
> 
> Moved following interface methods to ViewAndroid:
>  - onBackgroundColorChanged
>  - onTopControlsChanged
>  - onBottomControlsChanged
>  - onStartContentIntent
> 
> Remove following as they are not used:
>  - getSystemWindowInsetTop
>  - getSystemWindowInsetLeft
>  - getSystemWindowInsetRight
> 
> BUG=620172
> 
> Review-Url: https://codereview.chromium.org/2682593002
> Cr-Commit-Position: refs/heads/master@{#449186}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d95ac3f4aad80e2102fd8cf957dc...

TBR=boliu@chromium.org,dtrainor@chromium.org,jinsukkim@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=620172

Review-Url: https://codereview.chromium.org/2694273002
Cr-Commit-Position: refs/heads/master@{#450616}
Committed:
https://chromium.googlesource.com/chromium/src/+/cefe29f6134013119b32e9e96ffa...
==========

[commit-bot: I haz the power, 2017-02-15 07:01:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/cefe29f6134013119b32e9e96ffa...