| 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/connection_health_checker.h
|
| a794cd60a7339d576ea2eed263a4f0a20fb255af |
|
16-Jun-2015 |
Paul Stewart <pstew@chromium.org> |
shill: Top-level files a-e: Switch pointer/ref spacing
Switch to Chrome style spacing.
BUG=chromium:501026
TEST=Unit tests
Change-Id: Idb7252202e142098999fe0672712aded7d369259
Reviewed-on: https://chromium-review.googlesource.com/278038
Reviewed-by: Rebecca Silberstein <silberst@chromium.org>
Commit-Queue: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| 8d6b59704591ba9fad57751858835dc332dbdd37 |
|
28-Oct-2014 |
Peter Qiu <zqiu@chromium.org> |
shill: export a shared library for RTNL interface.
Export a shared library "libshill-net-<(libbase-ver)" from shill, which
currently includes the RTNL interface and other codes that it depends on
(mainly ByteString, IPAddress, IOHandler, TimeStamp, and Sockets). All the
library codes are moved to under "shill/net/".
Upcoming changes:
- Remove dependency for "shill/error.h" which depends on dbus-c++.
- Add Netlink interface to the shared library.
BUG=chromium:427982
TEST=unittests
CQ-DEPEND=CL:226103
Change-Id: I7f5c183b8a347c02e80ffed5e3497f691dde517c
Reviewed-on: https://chromium-review.googlesource.com/226110
Reviewed-by: Peter Qiu <zqiu@chromium.org>
Tested-by: Peter Qiu <zqiu@chromium.org>
Commit-Queue: Peter Qiu <zqiu@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| 22f1fbc11b69ee41af8370ec38f1b90577db6c3c |
|
17-Oct-2014 |
Ben Chan <benchan@chromium.org> |
shill: connection: Replace scoped_ptr with std::unique_ptr.
BUG=None
TEST=`USE='wimax' FEATURES=test emerge-$BOARD shill`
TEST=`USE='wimax clang asan' FEATURES=test emerge-$BOARD shill`
Change-Id: I41a34f47ce3998d380912c565a72e54e240d4436
Reviewed-on: https://chromium-review.googlesource.com/224159
Reviewed-by: Paul Stewart <pstew@chromium.org>
Commit-Queue: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/connection_health_checker.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/connection_health_checker.h
|
| 7fab89734d88724a288e96a9996b15548c5294c7 |
|
11-Aug-2014 |
Ben Chan <benchan@chromium.org> |
shill: Use integer types from stdint.h
This CL replaces the deprecated int* and uint* types from
'base/basictypes.h' with the int*_t and uint*_t types from 'stdint.h'.
BUG=chromium:401356
TEST=`USE='cellular gdmwimax wimax' FEATURES=test emerge-$BOARD platform2`
Change-Id: I3d4c195881203dd2a47dbb5af150b6c90b9c206e
Reviewed-on: https://chromium-review.googlesource.com/211770
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| a41ab517725d036b63420f8445550246f8f50b99 |
|
23-Jul-2014 |
Alex Vakulenko <avakulenko@chromium.org> |
shill: Sort headers alphabetically (build/include_alpha)
We are going to enable build/include_alpha linter warning soon,
so in preparation for this, fixed the warnings in shill.
BUG=None
TEST=cpplint.py --filter=-build/include_order,+build/include_alpha shill/*
Change-Id: I113ed7f9dd3f465a3f660599320cdb434e7cc214
Reviewed-on: https://chromium-review.googlesource.com/209636
Tested-by: Alex Vakulenko <avakulenko@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
/system/connectivity/shill/connection_health_checker.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/connection_health_checker.h
|
| 81404c674c82f0ac2c958715b451b84521148b7e |
|
09-May-2013 |
Prathmesh Prabhu <pprabhu@chromium.org> |
Test polling behaviour of ConnectionHealthChecker::VerifySentData
VerifySentData polls /proc/tcpinfo to verify data sent over a given
connection is actually delivered. Add unit-test where it polls twice.
BUG=chromium:234844
TEST=build and run unit-tests
Change-Id: I09c86f5771089521670a5ffbb39d2a3162c4fe3e
Reviewed-on: https://gerrit.chromium.org/gerrit/50544
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| 3fd54bd6e0c9bb81f4324cf96b53d35e7f134813 |
|
09-May-2013 |
Prathmesh Prabhu <pprabhu@chromium.org> |
Clean up ConnectionHealthChecker::Result
BUG=chromium:234734
TEST=build and run unit-tests
Change-Id: I2c70fd5f00f3e4795f75e6fd5648c94ae503e42b
Reviewed-on: https://gerrit.chromium.org/gerrit/50523
Reviewed-by: Thieu Le <thieule@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| ba99b598d3f399a41e57f49dccac5f988e653126 |
|
18-Apr-2013 |
Prathmesh Prabhu <pprabhu@chromium.org> |
shill: Create an IPAddress store in Manager for ConnectionHealthChecker
(1) Added class IPAddressStore
(2) Added an IPAddressStore object to Manager
(3) ConnectionHealthChecker uses this object so that the store persists
across Device resets.
BUG=chromium:232883
TEST=(1)build and run unit-tests.
(2)Check that DHCP renewal works -- connect the device to the same
wifi network twice and verify wifi is connected.
Change-Id: Ia5ee314db46c3de2e037f59c1f99a248c80e4ef9
Reviewed-on: https://gerrit.chromium.org/gerrit/49487
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| 41a071e432b09ab1945f5c9ae74c998726142810 |
|
26-Apr-2013 |
Paul Stewart <pstew@chromium.org> |
Revert "shill: Create an IPAddress store in Manager for ConnectionHealthChecker". This breaks all DHCP networks.
This reverts commit d8468b4fc2a7e071f3d4aff05b65ac8aa84ebdd4
Change-Id: Ic6ab705a6699039336e2407a87e48180b0cefffb
Reviewed-on: https://gerrit.chromium.org/gerrit/49316
Reviewed-by: Philipp Neubeck <pneubeck@chromium.org>
Commit-Queue: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| d8468b4fc2a7e071f3d4aff05b65ac8aa84ebdd4 |
|
18-Apr-2013 |
Prathmesh Prabhu <pprabhu@chromium.org> |
shill: Create an IPAddress store in Manager for ConnectionHealthChecker
(1) Added class IPAddressStore
(2) Added an IPAddressStore object to Manager
(3) ConnectionHealthChecker uses this object so that the store persists
across Device resets.
BUG=chromium:232883
TEST=build and run unit-tests.
Change-Id: I0d4f6de7c2c4d3b92bff76a4de3c663a42b9389f
Reviewed-on: https://gerrit.chromium.org/gerrit/48445
Reviewed-by: Arman Uguray <armansito@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| ade9b9a914efaeac3b2f4d4888ffdfb3e76b76f6 |
|
23-Apr-2013 |
Prathmesh Prabhu <pprabhu@chromium.org> |
shill: Make multiple TCP connections in ConnectionHealthChecker and
decide based on the whole sample
BUG=chromium:232136
TEST=(1) Build and run unit-tests
(2) Test that there aren't many false positives in out-of-credit
detection.
Change-Id: I3fa9d926536a2085dcdd8e6f7f92363e437a0b9f
Reviewed-on: https://gerrit.chromium.org/gerrit/48860
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
/system/connectivity/shill/connection_health_checker.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/connection_health_checker.h
|
| 5489b7a5b2f76725f4016846d57e1bad12e046e4 |
|
10-Apr-2013 |
Prathmesh Prabhu <pprabhu@chromium.org> |
shill: Add function to pretty-print result from ConnectionHealthChecker.
BUG=chromium:229982
TEST=build and run unit-tests.
Change-Id: I19d0b29bca278d28d7521871e03f0c1097f821d6
Reviewed-on: https://gerrit.chromium.org/gerrit/47789
Reviewed-by: Arman Uguray <armansito@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| f84a4242b4218dc375449ab2d68085226f43ce5b |
|
10-Apr-2013 |
Arman Uguray <armansito@chromium.org> |
shill: cellular: Use ConnectionHealthChecker for out-of-credit detection
Added code to trigger active connection monitoring using
ConnectionHealthChecker to detect out-of-credit scenarios, when:
1. TrafficMonitor detects congestion and thus a possible out-of-credit
situation;
2. Portal detection fails after Cellular connects.
BUG=chromium:225912,225915
TEST=1. Build and run unit tests.
2. While using a link device, run it out of credit. An
out-of-credit dialog should pop up and the service should
disconnect,
3. While on a link device with an out-of-credit SIM, try to connect
to Cellular. If it successfully connects and the connection
does not drop, an out-of-credit dialog should pop up and the
service should disconnect.
Change-Id: If70069c3630757b74099a4cefdfe6c67d8d308de
Reviewed-on: https://gerrit.chromium.org/gerrit/47717
Reviewed-by: Thieu Le <thieule@chromium.org>
Commit-Queue: Arman Uguray <armansito@chromium.org>
Tested-by: Arman Uguray <armansito@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| 40daa01c4c8d772889d9c4167dee312869152174 |
|
03-Apr-2013 |
Prathmesh Prabhu <pprabhu@chromium.org> |
shill: Add ConnectionHealthChecker class to check connectivity status.
ConnectionHealthChecker asynchronously creates a TCP connection with a
remote server and checks that either (1) a small amount of data can be
sent over the connection, or (2) the connection can be cleanly shut
down. ConnectionHealthChecker may be provided multiple remote server
addresses, and it attempts them in a round-robin fashion if one attempt
fails.
BUG=chromium:225913
TEST=Build and run unit tests.
Change-Id: I01013af2214fd5614b1fca820670de12757a8904
Reviewed-on: https://gerrit.chromium.org/gerrit/47752
Commit-Queue: Ben Chan <benchan@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| e0da0faa23c0c3dd600df28937f77d21b04a0b7b |
|
10-Apr-2013 |
Katie Roberts-Hoffman <katierh@chromium.org> |
Revert "shill: Add ConnectionHealthChecker class to check connectivity status."
Failure at http://chromegw/i/chromeos/builders/stout%20canary/builds/1178
This reverts commit bb67d87184c165028eb4a19699c439ba995422ef
Change-Id: Ia587da7ac45ba4f4ac368c16e3c9a02abb0d4161
Reviewed-on: https://gerrit.chromium.org/gerrit/47712
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Commit-Queue: Katie Roberts-Hoffman <katierh@chromium.org>
Reviewed-by: Katie Roberts-Hoffman <katierh@chromium.org>
Tested-by: Katie Roberts-Hoffman <katierh@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|
| bb67d87184c165028eb4a19699c439ba995422ef |
|
03-Apr-2013 |
Prathmesh Prabhu <pprabhu@chromium.org> |
shill: Add ConnectionHealthChecker class to check connectivity status.
ConnectionHealthChecker asynchronously creates a TCP connection with a
remote server and checks that either (1) a small amount of data can be
sent over the connection, or (2) the connection can be cleanly shut
down. ConnectionHealthChecker may be provided multiple remote server
addresses, and it attempts them in a round-robin fashion if one attempt
fails.
BUG=chromium:225913
TEST=Build and run unit tests.
Change-Id: I38d9c107f4dd82ea780ee628d0b8bf4ec4929cf0
Reviewed-on: https://gerrit.chromium.org/gerrit/47248
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
/system/connectivity/shill/connection_health_checker.h
|