[Mac] Refactor the fullscreen menubar

base/mac/mac_util.mm

Index: base/mac/mac_util.mm
diff --git a/base/mac/mac_util.mm b/base/mac/mac_util.mm
index 2f54b2799b5b01932b94fce98a63e5145ebae94b..5aaaa3cc462328b4617925888c255604d630b976 100644
--- a/base/mac/mac_util.mm
+++ b/base/mac/mac_util.mm
@@ -173,6 +173,10 @@ CGColorSpaceRef GetSystemColorSpace() {
   return g_system_color_space;
 }
 
+bool IsCGFloatEqual(CGFloat a, CGFloat b) {
+  return fabs(a - b) <= std::numeric_limits<CGFloat>::epsilon();

[Mark Mentovai, 2016/10/19 02:36:09]

#include <cmath> and use std::fabs() or std::abs() instead of this.

fabs always operates on and returns doubles. The namespaced variants operate on
the type of their argument, so they’ll be correct regardless of what CGFloat
turns out to be.

[spqchan, 2016/10/19 03:46:12]

Done.

+}
+
 // Add a request for full screen mode.  Must be called on the main thread.
 void RequestFullScreen(FullScreenMode mode) {
   DCHECK_LT(mode, kNumFullScreenModes);