[Material][Mac] Reduced the area above tabs by 2 pts

[Material][Mac] Reduced the area above tabs by 2 pts

BUG=610185
Committed: https://crrev.com/23202f9294a90af7fd7889856a2b546f23a845bc
Cr-Commit-Position: refs/heads/master@{#393940}

[spqchan, 2016-05-13 21:50:42]

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

[spqchan, 2016-05-13 21:50:43]

PTAL

[shrike, 2016-05-13 21:55:53]

lgtm

[spqchan, 2016-05-13 21:57:19]

The CQ bit was unchecked by spqchan@chromium.org

[spqchan, 2016-05-13 21:57:21]

The CQ bit was checked by spqchan@chromium.org

[spqchan, 2016-05-13 21:57:34]

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

[spqchan, 2016-05-13 21:57:34]

+avi for OWNERS

[commit-bot: I haz the power, 2016-05-13 21:57:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1971343004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1971343004/20001

[Avi (use Gerrit), 2016-05-13 22:01:12]

LGTM with fix.

https://codereview.chromium.org/1971343004/diff/20001/chrome/browser/ui/cocoa...
File chrome/browser/ui/cocoa/browser_window_layout.h (right):

https://codereview.chromium.org/1971343004/diff/20001/chrome/browser/ui/cocoa...
chrome/browser/ui/cocoa/browser_window_layout.h:15: extern CGFloat
TabStripHeight();
If this is a function, no "extern" on it. You need extern for constants only.

[spqchan, 2016-05-13 22:08:38]

https://codereview.chromium.org/1971343004/diff/20001/chrome/browser/ui/cocoa...
File chrome/browser/ui/cocoa/browser_window_layout.h (right):

https://codereview.chromium.org/1971343004/diff/20001/chrome/browser/ui/cocoa...
chrome/browser/ui/cocoa/browser_window_layout.h:15: extern CGFloat
TabStripHeight();
On 2016/05/13 22:01:12, Avi wrote:
> If this is a function, no "extern" on it. You need extern for constants only.

Done.

[spqchan, 2016-05-13 22:08:47]

The CQ bit was checked by spqchan@chromium.org

[spqchan, 2016-05-13 22:08:48]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
shrike@chromium.org
Link to the patchset: https://codereview.chromium.org/1971343004/#ps40001
(title: "Fix for avi")

[commit-bot: I haz the power, 2016-05-13 22:09:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1971343004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1971343004/40001

[commit-bot: I haz the power, 2016-05-13 23:00:11]

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

[commit-bot: I haz the power, 2016-05-13 23:00:12]

Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[spqchan, 2016-05-16 20:52:06]

The CQ bit was checked by spqchan@chromium.org

[spqchan, 2016-05-16 20:52:07]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
shrike@chromium.org
Link to the patchset: https://codereview.chromium.org/1971343004/#ps60001
(title: "Fixed test")

[commit-bot: I haz the power, 2016-05-16 20:52:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1971343004/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1971343004/60001

[commit-bot: I haz the power, 2016-05-16 21:37:51]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-16 21:38:57]

Description was changed from

==========
[Material][Mac] Reduced the area above tabs by 2 pts

BUG=610185
==========

to

==========
[Material][Mac] Reduced the area above tabs by 2 pts

BUG=610185

Committed: https://crrev.com/23202f9294a90af7fd7889856a2b546f23a845bc
Cr-Commit-Position: refs/heads/master@{#393940}
==========

[commit-bot: I haz the power, 2016-05-16 21:38:58]

Patchset 4 (id:??) landed as
https://crrev.com/23202f9294a90af7fd7889856a2b546f23a845bc
Cr-Commit-Position: refs/heads/master@{#393940}