Alternate NTP: Resize devtools when hiding bookmark bar

Alternate NTP: Resize devtools when hiding bookmark bar

Hiding the bookmark bar didn't update the devtools height. Fix was to update the frame when setTopContentOffset: was called.

BUG=173205
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182499

[dhollowa, 2013-02-13 23:31:30]

lgtm

https://codereview.chromium.org/12254021/diff/1/chrome/browser/ui/cocoa/dev_t...
File chrome/browser/ui/cocoa/dev_tools_controller_browsertest.mm (right):

https://codereview.chromium.org/12254021/diff/1/chrome/browser/ui/cocoa/dev_t...
chrome/browser/ui/cocoa/dev_tools_controller_browsertest.mm:112: CGFloat
new_offset = 25;
Hmm, snake-case is not obj-c style, but seems to be everywhere in this file, so
ok.

[sail, 2013-02-13 23:49:09]

https://codereview.chromium.org/12254021/diff/1/chrome/browser/ui/cocoa/dev_t...
File chrome/browser/ui/cocoa/dev_tools_controller_browsertest.mm (right):

https://codereview.chromium.org/12254021/diff/1/chrome/browser/ui/cocoa/dev_t...
chrome/browser/ui/cocoa/dev_tools_controller_browsertest.mm:112: CGFloat
new_offset = 25;
On 2013/02/13 23:31:30, dhollowa wrote:
> Hmm, snake-case is not obj-c style, but seems to be everywhere in this file,
so
> ok.

My understanding is that camel case is for code in Objective-C classes. Code in
C++ classes or functions should just snake-case.

[commit-bot: I haz the power, 2013-02-13 23:52:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sail@chromium.org/12254021/1

[commit-bot: I haz the power, 2013-02-14 18:02:48]

Retried try job too often on win7_aura for step(s) ash_unittests,
aura_unittests, compositor_unittests, content_browsertests, content_unittests,
interactive_ui_tests, unit_tests, views_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...