summaryrefslogtreecommitdiffstats
path: root/components/variations/variations_seed_store.cc
Commit message (Collapse)AuthorAgeFilesLines
* Switch SignatureVerifier to taking an algorithm enum.davidben2016-03-031-20/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The existing API and implementation were problematic for several reasons. - It is very unclear what algorithms were supported. - Everyone was using it as an enum anyway, but it required copy-and-pasting giant strings all over the codebase. - The API is dangerous. Anyone not using it as an enum (i.e. taking an AlgorithmIdentifier from another source) opens themselves up to accepting any random algorithm and parameters the underlying implementation knew how to parse. - It relies on EVP_get_digestbyobj extracting the hash for RSA-PKCS1-FOO signature OIDs. This is weird and, for EVP_get_digestbyobj, Chromium appears to be one of the only two consumers still relying on this. This is a remnant of OpenSSL's old EVP_Sign* APIs. - The old EVP_get_digestbyobj implementation failed to check that ECDSA public keys weren't used for an RSA algorithm, etc. - The old EVP_get_digestbyobj implementation happily accepted OIDs for hashes as signature algorithm OIDs. This removes a use of openssl/x509.h from //crypto. BUG=499653 Review URL: https://codereview.chromium.org/1679873005 Cr-Commit-Position: refs/heads/master@{#379014}
* Update components for new prefs location.brettw2016-02-011-2/+2
| | | | | | | | | | | | | | | | This is a search-and-replace update for includes with "base/prefs" -> "components/prefs" and the headers re-sorted. DEPS files were updated to allow the components in question to depend on components/prefs. This should be a no-op from a build perspective. BUG=583034 Review URL: https://codereview.chromium.org/1651203002 Cr-Commit-Position: refs/heads/master@{#372780}
* Move components/compression to third_party/zlib/googlehajimehoshi2016-01-141-1/+1
| | | | | | | | | | | | | | | | We are now implementing CompressibleString in third_party/WebKit/Source/ wtf/text and want to use compression algorithm, but wtf/text can't depend on components. This CL moves components/compression to third_party and enables wtf/text to use the compression utility. The purpose of this CL is similar to crrev.com/14021015. BUG=574317 TEST=n/a Review URL: https://codereview.chromium.org/1564773002 Cr-Commit-Position: refs/heads/master@{#369491}
* Switch to standard integer types in components/, part 4 of 4.avi2015-12-251-13/+16
| | | | | | | | | BUG=138542 TBR=blundell@chromium.org Review URL: https://codereview.chromium.org/1549993003 Cr-Commit-Position: refs/heads/master@{#366870}
* Removed callbacks to JNI functions to simplify code structure since they are ↵agulenko2015-11-181-11/+9
| | | | | | | | | | | | no longer needed. Also added gzip compressed seed support and pulling the actual time of HTTP response. BUG=551029 Review URL: https://codereview.chromium.org/1438123002 Cr-Commit-Position: refs/heads/master@{#360292}
* Re-uploading reverted CL 1437833004 ↵agulenko2015-11-181-1/+14
| | | | | | | | | | | | | (https://codereview.chromium.org/1437833004/) with the fix. Implemented clearing Java prefs after pulling variations first run seed from Java to C++ side. TBR=asvitkine@google.com BUG=551029 Review URL: https://codereview.chromium.org/1452413002 Cr-Commit-Position: refs/heads/master@{#360279}
* Revert of Implemented clearing Java prefs after pulling variations first run ↵loyso2015-11-141-14/+1
| | | | | | | | | | | | | | | | | | | | | | | | | seed from Java to C++ side (patchset #8 id:140001 of https://codereview.chromium.org/1437833004/ ) Reason for revert: Crashes android gn build (component unittests) Original issue's description: > Implemented clearing Java prefs after pulling variations first run seed from Java to C++ side > > BUG=551029 > > Committed: https://crrev.com/683e65ca552aad24fd7794ca362a198103519d7f > Cr-Commit-Position: refs/heads/master@{#359649} TBR=asvitkine@chromium.org,holte@chromium.org,agulenko@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=551029 Review URL: https://codereview.chromium.org/1447823003 Cr-Commit-Position: refs/heads/master@{#359726}
* Implemented clearing Java prefs after pulling variations first run seed from ↵agulenko2015-11-131-1/+14
| | | | | | | | | | Java to C++ side BUG=551029 Review URL: https://codereview.chromium.org/1437833004 Cr-Commit-Position: refs/heads/master@{#359649}
* Add a histogram to record Variations first run seed import on Android.asvitkine2015-11-091-7/+25
| | | | | | | | BUG=551029 Review URL: https://codereview.chromium.org/1414033014 Cr-Commit-Position: refs/heads/master@{#358610}
* Variations seed is pulled from the Java application on the first launch of ↵agulenko2015-11-051-0/+35
| | | | | | | | | | Chrome BUG=551029 Review URL: https://codereview.chromium.org/1417503010 Cr-Commit-Position: refs/heads/master@{#358006}
* Support gzip-compressed seed data from the variations server.veranika2015-10-191-7/+40
| | | | | | | | | | | | | | | | | | Firstly, the client asks for gzip-compressed data by adding "gzip" to A-IM HTTP request header. Secondly, if "gzip" is present in IM HTTP response header, the client uncompresses data. Gzip compression could be used in combination with delta compression. A-IM and IM headers are defined in RFC 3229: http://tools.ietf.org/html/rfc3229#page-32 This CL re-adds components/variations/service tests to components_unittests. BUG=543271 Review URL: https://codereview.chromium.org/1404583004 Cr-Commit-Position: refs/heads/master@{#354788}
* Use correct namespace in variations componentblundell2015-09-041-2/+2
| | | | | | | | | | | | | The componentization of //chrome/browser/metrics/variations is complete. This CL changes all code in //components/variations to use the variations namespace instead of the chrome_variations namespace. BUG=516685 TBR=thakis Review URL: https://codereview.chromium.org/1324733002 Cr-Commit-Position: refs/heads/master@{#347361}
* Componentize VariationsSeedStore and its unittestblundell2015-08-251-0/+453
This code is shared with iOS. It has no //chrome-level dependencies, so componentization is trivial. BUG=516662 TBR=agl Review URL: https://codereview.chromium.org/1271123003 Cr-Commit-Position: refs/heads/master@{#345340}
generated by cgit v1.1 at 2024-06-10 09:02:01 +0000