summaryrefslogtreecommitdiffstats
path: root/components/ownership
Commit message (Collapse)AuthorAgeFilesLines
* Rename USE_NSS to USE_NSS_CERTS.davidben2015-04-175-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | USE_NSS is confusing because it's not actually the analog of USE_OPENSSL; it's the analog to USE_OPENSSL_CERTS. This is in preparation for the chimera build which will set USE_OPENSSL and USE_NSS(_CERTS). This CL was partially done automatically by the following command: git grep -l USE_NSS | xargs sed -i -e 's/defined(USE_NSS)/defined(USE_NSS_CERTS)/' The remaining were caught by the following command and fixed manually: git grep 'USE_NSS\([^_]\|$\)' Finally, the following command verified nothing in a separate repository was sensitive to this change: find . -name '*.cc' -o -name '*.h' | xargs grep 'USE_NSS\([^_]\|$\)' For now, the old name is still defined, but not used within Chromium. A follow-up CL will remove deprecated use_nss and USE_NSS #defines together which will then cause downstream churn. Though from a grep of known downstreams, the churn seems to be fairly minimal. The removal is split from this CL so that, if we need to revert, the CL to revert is small. TBR=pneubeck@chromium.org BUG=462040 Review URL: https://codereview.chromium.org/1082123003 Cr-Commit-Position: refs/heads/master@{#325710}
* Fix dependencies on //components/policy in gn filestmoniuszko2015-02-261-1/+6
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/952163002 Cr-Commit-Position: refs/heads/master@{#318246}
* Add more size_t -> int warning fixes for the win x64 GN build.dpranke2015-02-201-0/+2
| | | | | | | | | R=brettw@chromium.org BUG=354261 Review URL: https://codereview.chromium.org/941863002 Cr-Commit-Position: refs/heads/master@{#317439}
* Cleanup owners files, src code.nkostylev2015-02-171-1/+2
| | | | | | | | | | | | Remove ygorshenin@ from OWNERS/TODOs, replace with alemate/antrim/dpolukhin/bartfab BUG=None TBR=stevenjb@chromium.org,isherman@chromium.org Review URL: https://codereview.chromium.org/925963002 Cr-Commit-Position: refs/heads/master@{#316554}
* AccountsOptionsHandler uses OwnerSettingsService now.ygorshenin2014-12-251-0/+9
| | | | | | | | | BUG=433840 TEST=manual tests on Falco device Review URL: https://codereview.chromium.org/826703002 Cr-Commit-Position: refs/heads/master@{#309646}
* gn format //componentsscottmg2014-12-031-4/+4
| | | | | | | | | | | No intended functionality change. R=brettw@chromium.org BUG=348474 Review URL: https://codereview.chromium.org/780633002 Cr-Commit-Position: refs/heads/master@{#306640}
* Added key-value storage for pending changes to OwnerSettingsServiceChromeOS.ygorshenin2014-11-101-2/+2
| | | | | | | | | | | This CL is an alternative to the https://codereview.chromium.org/709763002/. BUG=430042 TEST=manual tests on a falco device Review URL: https://codereview.chromium.org/695193004 Cr-Commit-Position: refs/heads/master@{#303465}
* Standardize usage of virtual/override/final specifiers.dcheng2014-10-294-9/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Google C++ style guide states: Explicitly annotate overrides of virtual functions or virtual destructors with an override or (less frequently) final specifier. Older (pre-C++11) code will use the virtual keyword as an inferior alternative annotation. For clarity, use exactly one of override, final, or virtual when declaring an override. To better conform to these guidelines, the following constructs have been rewritten: - if a base class has a virtual destructor, then: virtual ~Foo(); -> ~Foo() override; - virtual void Foo() override; -> void Foo() override; - virtual void Foo() override final; -> void Foo() final; This patch was automatically generated. The clang plugin can generate fixit hints, which are suggested edits when it is 100% sure it knows how to fix a problem. The hints from the clang plugin were applied to the source tree using the tool in https://codereview.chromium.org/598073004. BUG=417463 R=caitkp@chromium.org Review URL: https://codereview.chromium.org/684513002 Cr-Commit-Position: refs/heads/master@{#301931}
* Implemented OwnerSettingsService::Set() method.ygorshenin2014-10-272-18/+94
| | | | | | | | | | | | BUG=230018 TEST=unit_tests:OwnerSettingsServiceChromeOSTest.* Committed: https://crrev.com/5fa0ee22683b92d67e07c060e8c7a3c4cced51ee Cr-Commit-Position: refs/heads/master@{#301132} Review URL: https://codereview.chromium.org/654263003 Cr-Commit-Position: refs/heads/master@{#301328}
* Revert of Implemented OwnerSettingsService::Set() method. (patchset #7 ↵avi2014-10-242-94/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | id:160001 of https://codereview.chromium.org/654263003/) Reason for revert: ASAN bot unhappy http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/3731 Original issue's description: > Implemented OwnerSettingsService::Set() method. > > BUG=230018 > TEST=unit_tests:OwnerSettingsServiceChromeOSTest.* > > Committed: https://crrev.com/5fa0ee22683b92d67e07c060e8c7a3c4cced51ee > Cr-Commit-Position: refs/heads/master@{#301132} TBR=mnissler@chromium.org,pastarmovj@chromium.org,ygorshenin@chromium.org NOTREECHECKS=true NOTRY=true BUG=230018 Review URL: https://codereview.chromium.org/640063008 Cr-Commit-Position: refs/heads/master@{#301161}
* Implemented OwnerSettingsService::Set() method.ygorshenin2014-10-242-18/+94
| | | | | | | | | BUG=230018 TEST=unit_tests:OwnerSettingsServiceChromeOSTest.* Review URL: https://codereview.chromium.org/654263003 Cr-Commit-Position: refs/heads/master@{#301132}
* Fix various chromeos GN issuesjamesr2014-10-231-1/+1
| | | | | | | | | R=cmasone@chromium.org BUG=424337 Review URL: https://codereview.chromium.org/652893004 Cr-Commit-Position: refs/heads/master@{#300863}
* Standardize usage of virtual/override/final in components/dcheng2014-10-213-7/+7
| | | | | | | | | BUG=417463 TBR=blundell@chromium.org Review URL: https://codereview.chromium.org/666133002 Cr-Commit-Position: refs/heads/master@{#300456}
* replace OVERRIDE and FINAL with override and final in components/mostynb2014-10-063-7/+7
| | | | | | | | BUG=417463 Review URL: https://codereview.chromium.org/623133002 Cr-Commit-Position: refs/heads/master@{#298237}
* Generalize crypto::SignatureCreator to allow choice of hash function, so as ↵dougsteed2014-09-191-1/+2
| | | | | | | | | | | | | | to support SHA256 (not just SHA1). BUG=412531 R=rsleevi@chromium.org,davidben@chromium.org TBR=pfeldman@chromium.org Review URL: https://codereview.chromium.org/560583002 Cr-Commit-Position: refs/heads/master@{#295747}
* Non-plafrom-specific part of an OwnerSettingsService is moved to ↵Yuri Gorshenin2014-09-154-0/+230
| | | | | | | | | | | | | | | | components/ownership/*. BUG=398856 TEST=existing browser_tests and unit_tests R=erg@chromium.org, jochen@chromium.org, nkostylev@chromium.org, pastarmovj@chromium.org Committed: https://chromium.googlesource.com/chromium/src/+/46a58bffbe303d3484d7cc288b47bfae5388109c Committed: https://chromium.googlesource.com/chromium/src/+/40979bf265c3ffbe1736a87935418da72be5d1f1 Review URL: https://codereview.chromium.org/548323003 Cr-Commit-Position: refs/heads/master@{#294810}
* Revert "Non-plafrom-specific part of an OwnerSettingsService is moved to ↵Yuri Gorshenin2014-09-154-229/+0
| | | | | | | | | | | | components/ownership/*." This reverts commit 40979bf265c3ffbe1736a87935418da72be5d1f1. BUG=398856 Review URL: https://codereview.chromium.org/570053002 Cr-Commit-Position: refs/heads/master@{#294794}
* Non-plafrom-specific part of an OwnerSettingsService is moved to ↵Yuri Gorshenin2014-09-154-0/+229
| | | | | | | | | | | | | | components/ownership/*. BUG=398856 TEST=existing browser_tests and unit_tests R=erg@chromium.org, jochen@chromium.org, nkostylev@chromium.org, pastarmovj@chromium.org Committed: https://chromium.googlesource.com/chromium/src/+/46a58bffbe303d3484d7cc288b47bfae5388109c Review URL: https://codereview.chromium.org/548323003 Cr-Commit-Position: refs/heads/master@{#294793}
* Add chrome test support target to GN build.Brett Wilson2014-09-121-0/+1
| | | | | | | | | | Instead of putting all the files in a bit test_support target, I made separate smaller test support targets for the various sub-projects (browser, common, etc.) and linked those in. This helps distribute the complexity. R=jamesr@chromium.org Review URL: https://codereview.chromium.org/559543003 Cr-Commit-Position: refs/heads/master@{#294641}
* Revert of Non-plafrom-specific part of an OwnerSettingsService is moved to ↵nasko2014-09-124-226/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | components/ownership/*. (patchset #4 id:60001 of https://codereview.chromium.org/548323003/) Reason for revert: Breaks compile on multiple bots: http://build.chromium.org/p/chromium.win/builders/Win%20Builder%20%28dbg%29/builds/3134 http://build.chromium.org/p/chromium.mac/builders/Mac%20Builder%20%28dbg%29/builds/73424 http://build.chromium.org/p/chromium.webkit/builders/GPU%20Win%20Builder%20%28dbg%29/builds/13365 Original issue's description: > Non-plafrom-specific part of an OwnerSettingsService is moved to components/ownership/*. > > BUG=398856 > TEST=existing browser_tests and unit_tests > R=erg@chromium.org, jochen@chromium.org, nkostylev@chromium.org, pastarmovj@chromium.org > > Committed: https://chromium.googlesource.com/chromium/src/+/46a58bffbe303d3484d7cc288b47bfae5388109c TBR=jochen@chromium.org,erg@chromium.org,nkostylev@chromium.org,pastarmovj@chromium.org,ygorshenin@chromium.org NOTREECHECKS=true NOTRY=true BUG=398856 Review URL: https://codereview.chromium.org/565293003 Cr-Commit-Position: refs/heads/master@{#294618}
* Non-plafrom-specific part of an OwnerSettingsService is moved to ↵Yuri Gorshenin2014-09-124-0/+226
| | | | | | | | | | | | components/ownership/*. BUG=398856 TEST=existing browser_tests and unit_tests R=erg@chromium.org, jochen@chromium.org, nkostylev@chromium.org, pastarmovj@chromium.org Review URL: https://codereview.chromium.org/548323003 Cr-Commit-Position: refs/heads/master@{#294609}
* Change base/file_utils.h includes to base/files/file_utils.h in chrome_elf/ ↵thestig2014-09-101-1/+1
| | | | | | | | | | and components/ Also fix the formatting of chrome_elf/OWNERS Review URL: https://codereview.chromium.org/546733002 Cr-Commit-Position: refs/heads/master@{#294245}
* Instantiation of OwnerKeyUtil is delegated to OwnerSettingsServiceFactory, ↵ygorshenin2014-09-051-13/+2
| | | | | | | | | | | as we're going to move as much as possible from OwnerSettingsService to components/ownership/* and because OwnerSettingsServiceFactory should do all platform-specific business. BUG=398856 TEST=existing browser_tests and unit_tests Review URL: https://codereview.chromium.org/516243002 Cr-Commit-Position: refs/heads/master@{#293486}
* OwnerKeyUtil is moved to components/ownership.ygorshenin2014-08-2911-0/+533
BUG=398856 TEST=existing browser_tests Review URL: https://codereview.chromium.org/494093002 Cr-Commit-Position: refs/heads/master@{#292616}
generated by cgit v1.1 at 2024-10-29 05:10:40 +0000