Make ui/gfx/range a component

ui/gfx/gfx.gyp

Index: ui/gfx/gfx.gyp
diff --git a/ui/gfx/gfx.gyp b/ui/gfx/gfx.gyp
index c648d303d18fd2c0935e0b2453bca433b8d66147..9406e8005fe4b0600826bd3da05c27665f524149 100644
--- a/ui/gfx/gfx.gyp
+++ b/ui/gfx/gfx.gyp
@@ -68,6 +68,32 @@
       ],
     },
     {
+      'target_name': 'gfx_range',
+      'type': '<(component)',
+      'dependencies': [
+        '<(DEPTH)/base/base.gyp:base',
+      ],
+      'defines': [
+        'GFX_IMPLEMENTATION',

[sky, 2016/04/27 16:51:30]

Don't you need a separate macro for the range files? Otherwise if something from
ui/gfx happens to include ui/gfx/range then you'll get the wrong define set.

[Fady Samuel, 2016/04/27 17:16:59]

Done.

+      ],
+      'sources': [
+        'range/range.cc',
+        'range/range.h',
+        'range/range_f.cc',
+        'range/range_f.h',
+        'range/range_mac.mm',
+        'range/range_win.cc',
+      ],
+      'conditions': [
+        ['OS=="win"', {
+          # TODO(jschuh): C4267: http://crbug.com/167187 size_t -> int
+          # C4324 is structure was padded due to __declspec(align()), which is
+          # uninteresting.
+          'msvs_disabled_warnings': [ 4267, 4324 ],
+        }],
+      ]
+    },
+    {
       'target_name': 'gfx',
       'type': '<(component)',
       'dependencies': [
@@ -82,6 +108,7 @@
         '<(DEPTH)/third_party/libpng/libpng.gyp:libpng',
         '<(DEPTH)/third_party/zlib/zlib.gyp:zlib',
         'gfx_geometry',
+        'gfx_range',
       ],
       # text_elider.h includes ICU headers.
       'export_dependent_settings': [
@@ -251,12 +278,6 @@
         'platform_font_mac.mm',
         'platform_font_win.cc',
         'platform_font_win.h',
-        'range/range.cc',
-        'range/range.h',
-        'range/range_f.cc',
-        'range/range_f.h',
-        'range/range_mac.mm',
-        'range/range_win.cc',
         'render_text.cc',
         'render_text.h',
         'render_text_harfbuzz.cc',