Move WriteInto to base namespace.
Update all callers.
De-inlines the template. For some reason touching the namespaces started givin errors about DCHECK not being defined. Indeed, this header does not include logging.h. Rather than include logging in this very common header, it seems preferable to move the implementation to the .cc file.
This removes support for the wstring variant and updates the callers that used it.
[email protected], ben
TBR=jschuh (sandbox_win.cc)
NOPRESUBMIT=true
(this patch touches code using legacy wstrings)
Review URL: https://codereview.chromium.org/1223983002.
Cr-Commit-Position: refs/heads/master@{#337703}
diff --git a/components/omnibox/search_provider.cc b/components/omnibox/search_provider.cc
index ad0259c..d70513a6 100644
--- a/components/omnibox/search_provider.cc
+++ b/components/omnibox/search_provider.cc
@@ -1467,7 +1467,7 @@
if (current_time > token_expiration_time_) {
const size_t kTokenBytes = 12;
std::string raw_data;
- base::RandBytes(WriteInto(&raw_data, kTokenBytes + 1), kTokenBytes);
+ base::RandBytes(base::WriteInto(&raw_data, kTokenBytes + 1), kTokenBytes);
base::Base64Encode(raw_data, ¤t_token_);
// Make the base64 encoded value URL and filename safe(see RFC 3548).
