Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(290)

Unified Diff: webrtc/base/helpers.cc

Issue 1426443007: Revert of Adding the ability to create an RtpSender without a track. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/base/helpers.h ('k') | webrtc/base/helpers_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/base/helpers.cc
diff --git a/webrtc/base/helpers.cc b/webrtc/base/helpers.cc
index 1ad5d0e12ba9ae55ba302e3965f790e08744e050..8e59b6410c580b1721a689aee33bc33d758983c6 100644
--- a/webrtc/base/helpers.cc
+++ b/webrtc/base/helpers.cc
@@ -164,20 +164,16 @@
int seed_;
};
+// TODO: Use Base64::Base64Table instead.
+static const char BASE64[64] = {
+ 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M',
+ 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z',
+ 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm',
+ 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z',
+ '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '+', '/'
+};
+
namespace {
-
-// TODO: Use Base64::Base64Table instead.
-static const char kBase64[64] = {
- 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M',
- 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z',
- 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm',
- 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z',
- '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '+', '/'};
-
-static const char kHex[16] = {'0', '1', '2', '3', '4', '5', '6', '7',
- '8', '9', 'a', 'b', 'c', 'd', 'e', 'f'};
-
-static const char kUuidDigit17[4] = {'8', '9', 'a', 'b'};
// This round about way of creating a global RNG is to safe-guard against
// indeterminant static initialization order.
@@ -236,48 +232,13 @@
}
bool CreateRandomString(size_t len, std::string* str) {
- return CreateRandomString(len, kBase64, 64, str);
+ return CreateRandomString(len, BASE64, 64, str);
}
bool CreateRandomString(size_t len, const std::string& table,
std::string* str) {
return CreateRandomString(len, table.c_str(),
static_cast<int>(table.size()), str);
-}
-
-// Version 4 UUID is of the form:
-// xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx
-// Where 'x' is a hex digit, and 'y' is 8, 9, a or b.
-std::string CreateRandomUuid() {
- std::string str;
- scoped_ptr<uint8_t[]> bytes(new uint8_t[31]);
- if (!Rng().Generate(bytes.get(), 31)) {
- LOG(LS_ERROR) << "Failed to generate random string!";
- return str;
- }
- str.reserve(36);
- for (size_t i = 0; i < 8; ++i) {
- str.push_back(kHex[bytes[i] % 16]);
- }
- str.push_back('-');
- for (size_t i = 8; i < 12; ++i) {
- str.push_back(kHex[bytes[i] % 16]);
- }
- str.push_back('-');
- str.push_back('4');
- for (size_t i = 12; i < 15; ++i) {
- str.push_back(kHex[bytes[i] % 16]);
- }
- str.push_back('-');
- str.push_back(kUuidDigit17[bytes[15] % 4]);
- for (size_t i = 16; i < 19; ++i) {
- str.push_back(kHex[bytes[i] % 16]);
- }
- str.push_back('-');
- for (size_t i = 19; i < 31; ++i) {
- str.push_back(kHex[bytes[i] % 16]);
- }
- return str;
}
uint32_t CreateRandomId() {
« no previous file with comments | « webrtc/base/helpers.h ('k') | webrtc/base/helpers_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698