[Mac] Refactor the Fullscreen Toolbar

[Mac] Refactor the Fullscreen Toolbar

- Split the FullscreenToolbarController into a layout manager with
multiple trackers
- Wrote tests

[spqchan, 2016-10-18 22:29:35]

Description was changed from

==========
[Mac] Fullscreen Toolbar Refactoring

in progress so no review pls :D

BUG=
==========

to

==========
[Mac] Refactor the Fullscreen Toolbar

- Split the FullscreenToolbarController into a layout manager with
multiple trackers
- Wrote tests
==========

[spqchan, 2016-10-18 22:29:35]

spqchan@chromium.org changed reviewers:
+ erikchen@chromium.org

[spqchan, 2016-10-18 22:34:51]

This CL is a bit big, so I'm thinking of splitting it up to make things easier.

I'm thinking of putting each tracking class in a separate CL and then have a
final CL will replace FullscreenToolbarController with
FullscreenToolbarLayoutManager.

If you're okay with not splitting it up though, I can leave it. Thoughts?

[erikchen, 2016-10-18 23:20:15]

On 2016/10/18 22:34:51, spqchan wrote:
> This CL is a bit big, so I'm thinking of splitting it up to make things
easier.
> 
> I'm thinking of putting each tracking class in a separate CL and then have a
> final CL will replace FullscreenToolbarController with
> FullscreenToolbarLayoutManager.
> 
> If you're okay with not splitting it up though, I can leave it. Thoughts?

I would really appreciate it if you split this up. I can review this CL if you
want, but I can't guarantee a thorough review of a CL of this magnitude.

[spqchan, 2016-10-18 23:42:46]

On 2016/10/18 23:20:15, erikchen wrote:
> On 2016/10/18 22:34:51, spqchan wrote:
> > This CL is a bit big, so I'm thinking of splitting it up to make things
> easier.
> > 
> > I'm thinking of putting each tracking class in a separate CL and then have a
> > final CL will replace FullscreenToolbarController with
> > FullscreenToolbarLayoutManager.
> > 
> > If you're okay with not splitting it up though, I can leave it. Thoughts?
> 
> I would really appreciate it if you split this up. I can review this CL if you
> want, but I can't guarantee a thorough review of a CL of this magnitude.

Sounds good, I'll split this up.