commit | a06100a9df9956cb3fe5bc161ffde9e8734a96f9 | [log] [tgz] |
---|---|---|
author | Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> | Thu Feb 14 19:28:54 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Thu Feb 14 19:28:54 2019 |
tree | e3fe793a951559bfc665c8d75be1f6d6d3d8a7a1 | |
parent | 24408b586ababa6c7b548b1141281839461e02dc [diff] |
FlexLayout: Use std::ceil() rather than std::ceilf() This fixes the build with libstdc++ after commit 3a6cdd45b3 ('Improvements in FlexLayout "stretch" cross-axis alignment handling'): ../../ui/views/layout/flex_layout_types_internal.cc: In member function ‘void views::internal::Span::Center(const views::internal::Span&, const views::internal::Inset1D&)’: ../../ui/views/layout/flex_layout_types_internal.cc:57:20: error: ‘ceilf’ is not a member of ‘std’ set_start(std::ceilf(remaining * 0.5f)); ^~~~~ ../../ui/views/layout/flex_layout_types_internal.cc:57:20: note: suggested alternative: ‘ceil’ set_start(std::ceilf(remaining * 0.5f)); ^~~~~ ceil ceilf() is not officially part of the C++14 standard even though libc++ exposes it. In this case, we're multiplying an int and a float, which means we'll call ceil(float) anyway. Bug: 819294 Change-Id: I24f16a5cd0462baef0eea07791758a3712ff2922 Reviewed-on: https://chromium-review.googlesource.com/c/1472673 Auto-Submit: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> Commit-Queue: Dana Fried <dfried@chromium.org> Reviewed-by: Dana Fried <dfried@chromium.org> Cr-Commit-Position: refs/heads/master@{#632320}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .