aea4c1cea20dda7ae7e85fc8924a2d784f70d806 |
|
20-Aug-2015 |
Alex Deymo <deymo@google.com> |
Re-license update_engine to Apache2 This patch automatically replaced the license on all text files from Chromium OS (BSD style) to AOSP (Apache2), keeping the original year as a reference. The license header was added to .gyp and .gypi files, the NOTICE was replaced with a copy of the Apache2 license and MODULE_LICENSE_* file was updated. BUG=b/23084294 TEST=grep 'Chromium OS Authors' doesn't find anything. Change-Id: Ie5083750755f5180a8a785b24fe67dbf9195cd10
/system/update_engine/fake_p2p_manager.h
|
610277efc6f7e5239158dfa4bb3b1021804326e0 |
|
12-Nov-2014 |
Alex Deymo <deymo@chromium.org> |
update_engine: Add override when possible. Google Style Guide requires to include the "override" keyword when overriding a method on a derived class, so the compiler will catch errors if the method is not overriding a member of the base class. This patch introduces the "override" keyword when possible. BUG=None TEST=FEATURES=test emerge-link update_engine Change-Id: Ie83d115c5730f3b35b3d95859a54bc1a48e0be7b Reviewed-on: https://chromium-review.googlesource.com/228928 Tested-by: Alex Deymo <deymo@chromium.org> Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Commit-Queue: Alex Deymo <deymo@chromium.org>
/system/update_engine/fake_p2p_manager.h
|
4a0321b804524e001b482edea882bd1fc4af98a2 |
|
28-Oct-2014 |
Gilad Arnold <garnold@chromium.org> |
update_engine: Policy determines P2P enabled status. This switches the P2P Manager to use the newly introduced Update Manger policy requests in determining whether P2P is enabled on the system. There is a policy request for the initial state (P2PEnabled, sync) and for tracking changes (P2PEnabledChanged, async), with the latest known value being cached by the P2P Manager. This also reverses a recent change that moved P2P prefs setting into the P2PManager. In the absence of any additional logic (now cleared) there was no point in having a dedicated method just for that, and so dbus_service writes the prefs value directly. This affords us removing the prefs argument when initializing the P2PManager. BUG=chromium:425233 TEST=Unit tests. Change-Id: I53280f05da8fe532b6502c175a8cc9ddc1e15a87 Reviewed-on: https://chromium-review.googlesource.com/226937 Tested-by: Gilad Arnold <garnold@chromium.org> Reviewed-by: David Zeuthen <zeuthen@chromium.org> Commit-Queue: Gilad Arnold <garnold@chromium.org>
/system/update_engine/fake_p2p_manager.h
|
ccd0957b359136aca2d4d5c7728535d8a2e33c51 |
|
27-Oct-2014 |
Gilad Arnold <garnold@chromium.org> |
update_engine: Move P2P enabled pref handling into P2PManager. This makes the P2PManager own the manipulation of the P2P enabled setting, which positions us better toward moving the decision about whether P2P is enabled from P2PManager and into the UpdateManager. Also increases encapsulation and makes it a bit more resilient. BUG=chromium:425233 TEST=Unit tests. Change-Id: Ic91872df84920ae80c5ef973aee99cc46cac264c Reviewed-on: https://chromium-review.googlesource.com/225681 Tested-by: Gilad Arnold <garnold@chromium.org> Reviewed-by: Gaurav Shah <gauravsh@chromium.org> Commit-Queue: Gilad Arnold <garnold@chromium.org>
/system/update_engine/fake_p2p_manager.h
|
cf175a098081f3f0e9ca52d997a7ce1585c14c2d |
|
11-Jul-2014 |
Gilad Arnold <garnold@chromium.org> |
Fix cpplint errors. The only non-obvious change here is the switch from dynamic_cast to static_cast in three cases of down-casting in UpdateAttempter. dynamic_cast is banned by style, nor does it add any safety in this particular case (subsequent code dereferences the result right away without checking whether it's null). BUG=None TEST=None Change-Id: I9d49b46362feaf9c6fa13b2715ebe9fe50308a9a Reviewed-on: https://chromium-review.googlesource.com/207470 Tested-by: Gilad Arnold <garnold@chromium.org> Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Commit-Queue: Gilad Arnold <garnold@chromium.org>
/system/update_engine/fake_p2p_manager.h
|
d2779df63aaad8b65fc5d4badee7dbc9bed7f2b6 |
|
16-Jun-2014 |
Alex Vakulenko <avakulenko@chromium.org> |
update_engine: fixed warnings from cpplint Fixed all the cpplint warnings in update engine. BUG=None TEST=Unit tests still pass. Change-Id: I285ae858eec8abe0b26ff203b99a42a200ceb71c Reviewed-on: https://chromium-review.googlesource.com/204027 Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Tested-by: Alex Vakulenko <avakulenko@chromium.org> Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
/system/update_engine/fake_p2p_manager.h
|
04f2b380d4707a3903098a7459443c9509745943 |
|
21-Mar-2014 |
Alex Deymo <deymo@chromium.org> |
linter: Includes should use the full path from the source directory. Per the Google Style Guide #includes of a project's header files should be listed as descendants of the project's source directory. http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Names_and_Order_of_Includes#Names_and_Order_of_Includes In update_engine, all the project's include have the update_engine/ directory prefix. BUG=None TEST=Build and unittests pass. Change-Id: I8ab300491f868723848f76c27a3d2de86598ba2f Reviewed-on: https://chromium-review.googlesource.com/191180 Reviewed-by: Chris Sosa <sosa@chromium.org> Commit-Queue: Alex Deymo <deymo@chromium.org> Tested-by: Alex Deymo <deymo@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org>
/system/update_engine/fake_p2p_manager.h
|
759c275760b51defcfe5545abb887ad2616335f4 |
|
18-Mar-2014 |
Alex Deymo <deymo@chromium.org> |
Fix header guards to comply with Google Coding Style. The Google Style Guide says that every header file should have a define guard and the format of the symbol name should be <PROJECT>_<PATH>_<FILE>_H_ This patch does all the minor fixes to comply with this and includes a header guard for the bzip.h file, which didn't have it. Also, the Copyright notice is adjusted to the Chromium OS code, replacing "Chromium Authors" by "Chromium OS Authors". BUG=None TEST=build passes. Change-Id: I6575cc307c464d60a5cb2b132cf1e46acb6500b5 Reviewed-on: https://chromium-review.googlesource.com/190445 Tested-by: Alex Deymo <deymo@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org> Commit-Queue: Alex Deymo <deymo@chromium.org>
/system/update_engine/fake_p2p_manager.h
|
92d9c8bc5cd418e34944d6fc4bae44e3b4370246 |
|
11-Sep-2013 |
David Zeuthen <zeuthen@chromium.org> |
Also use DevicePolicy::GetAuP2PEnabled() to determine if p2p is enabled With this change update_engine will use p2p either if the user has manually enabled it (through the crosh flag) OR if the enterprise has enabled it in Enterprise Policy. BUG=chromium:260442 TEST=New unit tests + unit tests pass. Change-Id: I54cd92c481bd2fd7c90232d7137ce2b37fa2ce61 Reviewed-on: https://chromium-review.googlesource.com/168950 Reviewed-by: Chris Sosa <sosa@chromium.org> Commit-Queue: David Zeuthen <zeuthen@chromium.org> Tested-by: David Zeuthen <zeuthen@chromium.org>
/system/update_engine/fake_p2p_manager.h
|
3ff6e6ec99106c997db7c352828a02d9bbacc663 |
|
06-Aug-2013 |
David Zeuthen <zeuthen@chromium.org> |
p2p: Add fake implementation of P2PManager This fake implementation is useful when testing code using the P2PManager interface. BUG=chromium:260426 TEST=Unit tests pass Change-Id: I204c399fd321298bcf9239ad9cf3bdfaf9e38552 Reviewed-on: https://chromium-review.googlesource.com/64825 Reviewed-by: David Zeuthen <zeuthen@chromium.org> Commit-Queue: David Zeuthen <zeuthen@chromium.org> Tested-by: David Zeuthen <zeuthen@chromium.org>
/system/update_engine/fake_p2p_manager.h
|