Treat empty BigInteger as 0 per spec

R=eroman@chromium.org, rsleevi@chromium.org Bug=373552 TEST=crypto/ BUG= Review URL: https://codereview.chromium.org/596623004 Cr-Commit-Position: refs/heads/master@{#299649}
280723148 · Oct 15, 2014 · d9341e6 · d9341e6
1 parent 3f11282
commit d9341e6
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/AUTHORS b/AUTHORS
@@ -468,6 +468,7 @@ Xinchao He <hexinchao@gmail.com>
 Xing Zhang <xzhang@adobe.com>
 Xu Samuel <samuel.xu@intel.com>
 Xuefei Ren <xrenishere@gmail.com>
+Xun Sun <xun.sun@intel.com>
 Yael Aharon <yael.aharon@intel.com>
 Yair Yogev <progame@chromium.org>
 Yang Gu <yang.gu@intel.com>

diff --git a/content/child/webcrypto/webcrypto_util.cc b/content/child/webcrypto/webcrypto_util.cc
@@ -22,7 +22,6 @@ namespace {
 bool BigIntegerToUint(const uint8_t* data,
                       unsigned int data_size,
                       unsigned int* result) {
-  // TODO(eroman): Fix handling of empty biginteger. http://crbug.com/373552
   if (data_size == 0)
     return false;