Update iOS deployment target to 7.0

Update iOS deployment target to 7.0

iOS 6 will no longer be supported.

Replaces deprecated sizeWithFont: with sizeWithAttributes: (adding a
ceilf since according to the internet, the former rounds up, and the
latter does not) to fix compilation.

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

[stuartmorgan, 2014-08-12 22:48:40]

Nico for gfx/ owners (and also because you'll probably be amused to know that
the C++ header bundling issue with clang is the only reason I knew doing this
switched us to libc++; otherwise who knows when we'd have realized)

[Nico, 2014-08-12 22:56:50]

lgtm

[stuartmorgan, 2014-08-12 22:58:10]

+Sylvain for a look at the gyp change since it's related to the libc++ work.

[sdefresne, 2014-08-13 09:30:37]

LGTM with nits

https://codereview.chromium.org/463333002/diff/1/ui/gfx/platform_font_ios.mm
File ui/gfx/platform_font_ios.mm (right):

https://codereview.chromium.org/463333002/diff/1/ui/gfx/platform_font_ios.mm#...
ui/gfx/platform_font_ios.mm:114: average_width_ = ceilf([@"x"
sizeWithAttributes:attributes].width);
nit: why not use std::ceil from <cmath> and avoid the conversion from CGFloat to
float to CGFloat on arm64?

https://codereview.chromium.org/463333002/diff/1/ui/gfx/text_utils_ios.mm
File ui/gfx/text_utils_ios.mm (right):

https://codereview.chromium.org/463333002/diff/1/ui/gfx/text_utils_ios.mm#new...
ui/gfx/text_utils_ios.mm:24: return ceilf([ns_text
sizeWithAttributes:attributes].width);
nit: same question about std::ceil?

[stuartmorgan, 2014-08-13 14:07:36]

https://codereview.chromium.org/463333002/diff/1/ui/gfx/platform_font_ios.mm
File ui/gfx/platform_font_ios.mm (right):

https://codereview.chromium.org/463333002/diff/1/ui/gfx/platform_font_ios.mm#...
ui/gfx/platform_font_ios.mm:114: average_width_ = ceilf([@"x"
sizeWithAttributes:attributes].width);
On 2014/08/13 09:30:37, sdefresne wrote:
> nit: why not use std::ceil from <cmath> and avoid the conversion from CGFloat
to
> float to CGFloat on arm64?

Good call; done.

https://codereview.chromium.org/463333002/diff/1/ui/gfx/text_utils_ios.mm
File ui/gfx/text_utils_ios.mm (right):

https://codereview.chromium.org/463333002/diff/1/ui/gfx/text_utils_ios.mm#new...
ui/gfx/text_utils_ios.mm:24: return ceilf([ns_text
sizeWithAttributes:attributes].width);
On 2014/08/13 09:30:37, sdefresne wrote:
> nit: same question about std::ceil?

Done.

[stuartmorgan, 2014-08-13 14:07:56]

The CQ bit was checked by stuartmorgan@chromium.org

[commit-bot: I haz the power, 2014-08-13 14:08:34]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/stuartmorgan@chromium.org/463333002/20001

[commit-bot: I haz the power, 2014-08-13 16:45:38]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tri...)

[commit-bot: I haz the power, 2014-08-13 17:07:57]

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

[commit-bot: I haz the power, 2014-08-13 17:07:59]

Try jobs failed on following builders:
  android_dbg_triggered_tests on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tri...)

[stuartmorgan, 2014-08-13 21:25:08]

The CQ bit was checked by stuartmorgan@chromium.org

[commit-bot: I haz the power, 2014-08-13 21:27:00]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/stuartmorgan@chromium.org/463333002/20001

[commit-bot: I haz the power, 2014-08-13 22:50:27]

Committed patchset #2 (20001) as 289403