c0beca55d290fe0b1c96d78cbbcf94b05c23f5a5 |
|
03-Sep-2015 |
Peter Qiu <zqiu@google.com> |
License shill 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, the NOTICE was added with a copy of the Apache2 license and MODULE_LICENSE_* file was added. BUG=23587594 TEST=grep 'Chromium OS Authors' doesn't find anything. Change-Id: If41ede586c2465f5831fb7fee270ff41dbfdb596
/system/connectivity/shill/mock_dns_client.h
|
1e006c610db980bca337121134d2ba54497b7a59 |
|
16-Jun-2015 |
Paul Stewart <pstew@chromium.org> |
shill: Top-level mocks: Switch pointer/ref spacing Switch to Chrome style spacing. BUG=chromium:501026 TEST=Unit tests Change-Id: I730ed5115e62319c2745dd6cef642e228b0fb66e Reviewed-on: https://chromium-review.googlesource.com/278036 Reviewed-by: Rebecca Silberstein <silberst@chromium.org> Commit-Queue: Paul Stewart <pstew@chromium.org> Tested-by: Paul Stewart <pstew@chromium.org>
/system/connectivity/shill/mock_dns_client.h
|
cc67c52a2c00f90e877971d552208dd99825d84e |
|
03-Sep-2014 |
Ben Chan <benchan@chromium.org> |
shill: Include base/macros.h instead of base/basictypes.h shill no longer uses the integer types from base/basictypes.h. It should simply include base/macros.h for the DISALLOW_COPY_AND_ASSIGN macro instead. BUG=None TEST=`USE='cellular wimax' FEATURES=test emerge-$BOARD platform2` Change-Id: I9b350c1bd9eff88e2c87de2ef8d607e90049e102 Reviewed-on: https://chromium-review.googlesource.com/216055 Reviewed-by: Paul Stewart <pstew@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/mock_dns_client.h
|
5ea763b83299b5fad76a87183fb39a74c2d3c61d |
|
13-Aug-2014 |
Ben Chan <benchan@chromium.org> |
shill: Add C++11 override specifier to destructors of derived classes. BUG=None TEST=`USE=wimax FEATURES=test emerge-$BOARD platform2` Change-Id: I88d4e29165869b26924cccb94ac940dc8961c312 Reviewed-on: https://chromium-review.googlesource.com/212267 Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/mock_dns_client.h
|
8a5322984f2d81bcbfd8d44c59747a11bd9b904b |
|
17-Jun-2014 |
Alex Vakulenko <avakulenko@chromium.org> |
shill: Fix most of warnings from cpplint Fix most cpplinter's warnings. On a few occasisions, the warnings produced were false-positives, so added explicit // NOLINT comment overrides. There were a few instances of disallowed non-const reference usage (reported as runtime/reference error by cpplint) for "DBus::Error &error" which should potentially be changed to pointers to comply with C++ coding style but I will let Shill owners do that since there is quite a few of those and the change isn't brain-dead simple... BUG=None TEST=platform2 still compiles and all unit tests pass. Change-Id: Ic2e31896aa13d20eeb1a85dee74f3db8cccfde2e Reviewed-on: https://chromium-review.googlesource.com/204164 Tested-by: Alex Vakulenko <avakulenko@chromium.org> Reviewed-by: mukesh agrawal <quiche@chromium.org> Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
/system/connectivity/shill/mock_dns_client.h
|
3e452f8c41b7983f1068a4b7229998c7e7de4334 |
|
11-Apr-2013 |
Prathmesh Prabhu <pprabhu@chromium.org> |
shill: Spawn mulitple DNS queries when a remote URL is added to ConnectionHealthChecker When ConnectionHealthChecker::AddRemoteURL is called, (1) Multiple DNS queries are made for the same URL. (2) All non-duplicate IP addresses returned are stored. (3) When trying to connection to remote hosts, IP addresses are chosen at random. BUG=chromium:229985 TEST=run and build unit-tests Change-Id: I3ac062bc27071ea13606ce59e0f4a2313e64716e Reviewed-on: https://gerrit.chromium.org/gerrit/47945 Commit-Queue: Ben Chan <benchan@chromium.org> Reviewed-by: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/mock_dns_client.h
|
bdb02e664f79f29b3bce9be4ca05c82d8ede697f |
|
23-Feb-2012 |
Paul Stewart <pstew@chromium.org> |
shill: Prepare DNSClient and HTTPRequest for destroy-on-callback Refactor both classes so that it's okay for the callback to destroy the object (i.e., do nothing to the object after the callback is called). As a part of this, clean up some of the callback semantics, so, for example, DNSClient callbacks are passed an Error reference and an IP Address instead of having to use a getter. Additionally remove the blemish where an immediate timeout in Start() both returned failure and called the callback. BUG=chromium-os:23318 TEST=Fixed unit tests, manual Change-Id: Ib7787a7aa6f7f3d00caa539d6b0221ff5f3d60b3 Reviewed-on: https://gerrit.chromium.org/gerrit/16435 Commit-Ready: Paul Stewart <pstew@chromium.org> Reviewed-by: Paul Stewart <pstew@chromium.org> Tested-by: Paul Stewart <pstew@chromium.org>
/system/connectivity/shill/mock_dns_client.h
|
f65320cc1c04ea9e09cc8656e87fe9912c601e9a |
|
13-Oct-2011 |
Paul Stewart <pstew@chromium.org> |
shill: Add http_proxy class The http_proxy adds a device/connection based proxy that guarantees to the caller that its HTTP request will go out a particular device's connection. DNS requests occur through a bound socket to this device and goes to DNS servers configured on this connection. HTTP requests will also be bound to this interface. This facility will be used by a number of peripheral bits including portal detection, activation and cashew. BUG=chromium-os:21664 TEST=New unit test. New (disabled) functional test, against which I can run "curl -x" and Chrome with manual proxy settings. Change-Id: I0d59bf0ae27d3538ef359f786742f5c2f1d6fef9 Reviewed-on: https://gerrit.chromium.org/gerrit/10165 Reviewed-by: Thieu Le <thieule@chromium.org> Tested-by: Paul Stewart <pstew@chromium.org> Commit-Ready: Paul Stewart <pstew@chromium.org>
/system/connectivity/shill/mock_dns_client.h
|