Bug 1565515 - Removed duplication between LengthOfEncodedBinary and CalcDecodedStringySize r=ttung,asuth

Differential Revision: https://phabricator.services.mozilla.com/D38173

--HG--
extra : moz-landing-system : lando

dom/indexedDB/Key.cpp

@@ -123,16 +123,12 @@ IDBResult<void, IDBSpecialValue::Invalid> Key::SetFromString(
 // |aPos| should point to the type indicator.
 // The returned length doesn't include the type indicator
 // or the terminator.
-static size_t LengthOfEncodedBinary(const unsigned char* aPos,
+// static
-                                    const unsigned char* aEnd) {
+uint32_t Key::LengthOfEncodedBinary(const EncodedDataType* aPos,
+                                    const EncodedDataType* aEnd) {
   MOZ_ASSERT(*aPos % Key::eMaxType == Key::eBinary, "Don't call me!");
-  auto iter = ++aPos;
+  const EncodedDataType* encodedSectionEnd;
-  for (; iter < aEnd && *iter != Key::eTerminator; ++iter) {
+  return CalcDecodedStringySize<uint8_t>(aPos + 1, aEnd, &encodedSectionEnd);
-    if (*iter & 0x80) {
-      iter++;
-    }
-  }
-  return iter - aPos;
 }
 
 IDBResult<void, IDBSpecialValue::Invalid> Key::ToLocaleBasedKey(

dom/indexedDB/Key.h

@@ -330,6 +330,9 @@ class Key {
       const EncodedDataType* aBegin, const EncodedDataType* aEnd,
       const EncodedDataType** aOutEncodedSectionEnd);
 
+  static uint32_t LengthOfEncodedBinary(const EncodedDataType* aPos,
+                                        const EncodedDataType* aEnd);
+
   template <typename T>
   static void DecodeAsStringy(const EncodedDataType* aEncodedSectionBegin,
                               const EncodedDataType* aEncodedSectionEnd,