Use MarginLayoutParams in ToolbarManager

Use MarginLayoutParams in ToolbarManager

The control container can have different parent views depending on
which flags are enabled, namely Chrome Home. This change removes the
assumption that particular layout params are for a FrameLayout and
instead uses the least common denominator -- MarginLayoutParams. This
should be fine in this case since the margin is all that is being
modified.

BUG=664985
Committed: https://crrev.com/91df9834a095460208c7bbf0fcd3747c18eab64d
Cr-Commit-Position: refs/heads/master@{#438900}

[mdjones, 2016-12-15 18:08:36]

mdjones@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[mdjones, 2016-12-15 18:08:37]

Read my code

[gone, 2016-12-15 18:56:48]

Read.

lgtm

[mdjones, 2016-12-15 19:17:58]

The CQ bit was checked by mdjones@chromium.org

[commit-bot: I haz the power, 2016-12-15 19:18:29]

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, 2016-12-15 19:53:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481829478362170, "parent_rev":
"4dc92ce7058280eb2e5a68b6db886cdca2265884", "commit_rev":
"483f4a8668fa6500a53368f03d8db431ee44b748"}

[commit-bot: I haz the power, 2016-12-15 19:54:24]

Description was changed from

==========
Use MarginLayoutParams in ToolbarManager

The control container can have different parent views depending on
which flags are enabled, namely Chrome Home. This change removes the
assumption that particular layout params are for a FrameLayout and
instead uses the least common denominator -- MarginLayoutParams. This
should be fine in this case since the margin is all that is being
modified.

BUG=664985
==========

to

==========
Use MarginLayoutParams in ToolbarManager

The control container can have different parent views depending on
which flags are enabled, namely Chrome Home. This change removes the
assumption that particular layout params are for a FrameLayout and
instead uses the least common denominator -- MarginLayoutParams. This
should be fine in this case since the margin is all that is being
modified.

BUG=664985

Review-Url: https://codereview.chromium.org/2579833002
==========

[commit-bot: I haz the power, 2016-12-15 19:54:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-15 19:56:32]

Description was changed from

==========
Use MarginLayoutParams in ToolbarManager

The control container can have different parent views depending on
which flags are enabled, namely Chrome Home. This change removes the
assumption that particular layout params are for a FrameLayout and
instead uses the least common denominator -- MarginLayoutParams. This
should be fine in this case since the margin is all that is being
modified.

BUG=664985

Review-Url: https://codereview.chromium.org/2579833002
==========

to

==========
Use MarginLayoutParams in ToolbarManager

The control container can have different parent views depending on
which flags are enabled, namely Chrome Home. This change removes the
assumption that particular layout params are for a FrameLayout and
instead uses the least common denominator -- MarginLayoutParams. This
should be fine in this case since the margin is all that is being
modified.

BUG=664985

Committed: https://crrev.com/91df9834a095460208c7bbf0fcd3747c18eab64d
Cr-Commit-Position: refs/heads/master@{#438900}
==========

[commit-bot: I haz the power, 2016-12-15 19:56:33]

Patchset 1 (id:??) landed as
https://crrev.com/91df9834a095460208c7bbf0fcd3747c18eab64d
Cr-Commit-Position: refs/heads/master@{#438900}