Updating OTS repo from https://github.com/khaledhosny/ots.git

third_party/ots/ots.gyp

+# Copyright (c) 2009 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+{
+  'variables': {
+    'chromium_code': 1,
+  },
+  'includes': [
+    'ots-common.gypi',
+  ],
+  'targets': [
+    {
+      'target_name': 'ots',
+      'type': 'static_library',
+      'sources': [
+        '<@(ots_sources)',
+      ],
+      'include_dirs': [
+        '../..',
+        '<@(ots_include_dirs)',
+      ],
+      'direct_dependent_settings': {
+        'include_dirs': [
+          '<@(ots_include_dirs)',
+        ],
+      },
+      'dependencies': [
+        '../brotli/brotli.gyp:brotli',
+        '../zlib/zlib.gyp:zlib',
+      ],
+      # TODO(jschuh): http://crbug.com/167187
+      'msvs_disabled_warnings': [
+        4267,
+        4334,
+        4800,
+      ],      

[jungshik at Google, 2014/12/11 19:11:06]

You didn't address my comment about 4800.  4800 is not covered by
http://crbug.com/167187. 
Please, add a separate comment about 4800 referring to the github bug you filed.


How about this? 

# TODO(jschuh): 4267, 4334 are 64-bit cleanness issue.  http://crbug.com/167187
# TODO(h.joshi) : Make 4800 go away. <url to git hub bug> 

[h.joshi, 2014/12/12 04:11:22]

I did replied to the same in comment
(https://codereview.chromium.org/658573004/#msg40)

Will update and add suggested description with link to
github bug in patch description.

[jungshik at Google, 2014/12/12 08:01:45]

I did NOT mean adding what I suggested to the CL description. I wanted you to
update the comment in THIS file because "TODO(jschuh): http://crbug.com/167187"
is NOT applicable to 4800. 

+    },
+  ],
+}