Move base64-urlsafe helpers to jwk.{cc,h}

content/child/webcrypto/jwk.cc

 // Copyright 2014 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.
 
 #include "jwk.h"
 
 #include <algorithm>
 #include <functional>
 #include <map>
 
+#include "base/base64.h"
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
+#include "base/stl_util.h"
 #include "base/strings/string_piece.h"
 #include "content/child/webcrypto/crypto_data.h"
 #include "content/child/webcrypto/status.h"
 #include "content/child/webcrypto/webcrypto_util.h"
 
 // TODO(eroman): The algorithm-specific logic in this file for AES and RSA
 // should be moved into the corresponding AlgorithmImplementation file. It
 // exists in this file to avoid duplication between OpenSSL and NSS
 // implementations.
 
@@ skipping 672 matching lines @@
       return "HS256";
     case blink::WebCryptoAlgorithmIdSha384:
       return "HS384";
     case blink::WebCryptoAlgorithmIdSha512:
       return "HS512";
     default:
       return NULL;
   }
 }
 
+bool Base64DecodeUrlSafe(const std::string& input, std::string* output) {
+  std::string base64EncodedText(input);

[Ryan Sleevi, 2014/08/27 17:22:15]

1) variable naming / style
2) This still seems to allow invalid Base64URL codings

I'm happy to leave 2 for a follow-up, but 1 should be fixed.

[eroman, 2014/08/27 18:00:57]

Good catch, Done.
I have added a TODO which links to https://codereview.chromium.org/504413004

+  std::replace(base64EncodedText.begin(), base64EncodedText.end(), '-', '+');
+  std::replace(base64EncodedText.begin(), base64EncodedText.end(), '_', '/');
+  base64EncodedText.append((4 - base64EncodedText.size() % 4) % 4, '=');
+  return base::Base64Decode(base64EncodedText, output);
+}
+
+std::string Base64EncodeUrlSafe(const base::StringPiece& input) {
+  std::string output;
+  base::Base64Encode(input, &output);
+  std::replace(output.begin(), output.end(), '+', '-');
+  std::replace(output.begin(), output.end(), '/', '_');
+  output.erase(std::remove(output.begin(), output.end(), '='), output.end());
+  return output;
+}
+
+std::string Base64EncodeUrlSafe(const std::vector<uint8_t>& input) {
+  const base::StringPiece string_piece(
+      reinterpret_cast<const char*>(vector_as_array(&input)), input.size());
+  return Base64EncodeUrlSafe(string_piece);
+}
+
 }  // namespace webcrypto
 
 }  // namespace content