Age | Commit message (Collapse) | Author |
|
* Use const reference type for read-only parameters.
Bug: 30407689
Test: build with WITH_TIDY=1
Change-Id: I36db541ef5b063bc59ef639097119dc488936d5a
|
|
am: df4d25c8e1
Change-Id: I43e0f9e6f1678906509227a570f7972fe3fd54ec
|
|
am: 677cf01e80
Change-Id: If32df654608a371e8807818391f7070c142a4d23
|
|
am: ff25df29d3
Change-Id: I1b14f5e6d6163c7bacc99b1ac79228077ccd884f
|
|
am: 8113669c0a
Change-Id: Ic07a447b5443bf0344d80c083a0a4e55779806aa
|
|
Change-Id: I25ab1eb79d24d9a31ce511e55f115fa0ee6b4dba
Signed-off-by: Bin Chen <pierr.chen@gmail.com>
|
|
Change-Id: Ibea4ea8bcfa36a1cfe98188407805346dca22e84
Signed-off-by: Bin Chen <pierr.chen@gmail.com>
|
|
Change-Id: I84b860aaf7e306f2e3b7bedd078c1c2baf837e3b
Signed-off-by: Bin Chen <pierr.chen@gmail.com>
|
|
Zero-parameter constructors should not be marked explicit.
Change-Id: Iefa4bebb9393eb7625ee9d62402e4bff9f412b98
Signed-off-by: Bin Chen <pierr.chen@gmail.com>
|
|
am: 40745eaf9c
Change-Id: I556347dda12f6d7d789721fbe7dbad5897fd9f2a
|
|
Replaced with PRODUCT_PACKAGES_ENG in brillo_base.mk
Change-Id: I45caf2d5eeb4045a0bb9818b8fea267acefe9ac2
|
|
am: 9f450c918a
* commit '9f450c918a39099a159384c5bc691d3e9937dafb':
Add check for the selected_service before calling GetProperties
Change-Id: I86191a1bd659df5ac03b787ae42e9e340c087c75
|
|
am: e477f1e5fd
* commit 'e477f1e5fd82dfcb545b9f57b98b0d3384ac75de':
Add check for the selected_service before calling GetProperties
Change-Id: I0272c6907d5e66a990021c18322d0b0f7f98d08d
|
|
There are two net interfaces wlan0 and eth0 for the picoimx board.
When trying to register the board to cloud through weave manger,
The weaved will have below segment fault. The root cause is after wlan0
claimed from shill, there will be only eth0 been hold in device__. And
there is no related selected_service_ been set for the eth0
backtrace:
00 pc 000225de /system/bin/weaved
01 pc 0002247d /system/bin/weaved
02 pc 00022699 /system/bin/weaved
03 pc 0005eb69 /system/lib/libweave.so (weave::privet::WifiBootstrapMa
nager::GetCurrentlyConnectedSsid() const+8)
04 pc 000553f1 /system/lib/libweave.so (weave::privet::PrivetHandler::
HandleInfo(base::DictionaryValue const&, weave::privet::
UserInfo const&, base::Ca)
05 pc 00058333 /system/lib/libweave.so (weave::privet::PrivetHandler::
HandleRequest(std::__1::basic_string<char, std::__1::char_traits<char>,
std::__1:)
Change-Id: I9403a75fb4e4f022312b288d86332d5e5cc76f9d
|
|
am: 0d57d94734
* commit '0d57d947348dfd819daeacb36c320883e860df43':
Remove hard coded gtest/gmock includes/libs.
Change-Id: I8a792399a29c4f82ecc37bdba6be0c8788a0547e
|
|
am: c641b5117d
* commit 'c641b5117d9667751ba6f650176adfdcb12ce077':
Remove hard coded gtest/gmock includes/libs.
Change-Id: Ic155ee331768d46d7ca6eccb89b17ee85621fffb
|
|
Tested that checkbuild passes on volantis.
Bug: http://b/16574165
Change-Id: I7bbbd8b4b926115dec2a90ff36dab32b65a35360
|
|
am: f636ab4829
* commit 'f636ab482934d783c45e7bfc5225bd375435b05e':
weaved: Remove handler for base.reboot command
Change-Id: Ic8cc3051dfb245c04af484ef824b499dd672c9a2
|
|
am: 94e850c87e
* commit '94e850c87e6061a2dec5db6eba125f3cd5ad2a78':
weaved: Remove handler for base.reboot command
Change-Id: I4f9cc33716d7118cd85396335686f562cbf1e803
|
|
Since libweave no longer has the 'base' trait, its 'reboot' and
'identify' commands are removed too.
Delete the command handler for 'base.reboot' or else weaved just
crashes at startup (with 'unknown command' error).
Change-Id: Ic5cba6bccfc4d8d652b7c67a30f1c5ed3f608fc2
|
|
ebd4538fe7
am: b1601f6b51
* commit 'b1601f6b51ad88045b9f3e083cf3fe65b63d8742':
weaved: Fix LocalAccessEnabled flags to reflect changes in libweave
Change-Id: I336871419ba2c7f48dcfb355a6114fc26ffd3075
|
|
am: ebd4538fe7
* commit 'ebd4538fe7ee5c9266412f4a05649ba542f27c43':
weaved: Fix LocalAccessEnabled flags to reflect changes in libweave
Change-Id: Icf884e8467b090f7b772ca3ee431ba808041a214
|
|
On the libweave side, local_discovery_enabled and local_pairing_enabled
settings have been consolidated under one: "local_access_enabled".
Must made similar fixes in weaved for it to compile and work correctly.
Change-Id: I6b28aae80b374508419a69ea1b9e6874d61dd12e
|
|
am: d67ba44
* commit 'd67ba4472671786dd45380515a4132bc57ce35e9':
weaved: Remove release/aquire semantic from scoped_ptr/unique_ptr
Change-Id: I2f8a5984b28fc0cb71022edd60ddf22fb56153c8
|
|
am: 1333c63
* commit '1333c63f8e0f69d7c426087502b729fa250144f7':
weaved: Remove release/aquire semantic from scoped_ptr/unique_ptr
Change-Id: I5fcdf381bb4be129e9afa054c9bc50f99ea8b063
|
|
Now that scoped_ptr is just a type alias to unique_ptr, there is no
longer a need to convert between the two by using .release() and
constructing the other with a raw pointer.
BUG: None
TEST: Built for dragonboard, all unit tests pass (except for update_engine,
but those tests failed before these changes too).
Change-Id: Ifcda9520231cf4770d65d84c80cee481eb95285c
|
|
am: 9563ba5
* commit '9563ba5a2cee5ef456d9dfb23e5a08cd0fe27c65':
weaved: Only build tests for Brillo eng builds.
Change-Id: I9b47bcb5c6d29976be027bdf870faec334768b44
|
|
am: 8e19eba
* commit '8e19eba94887ce18acab864f9e213781b1d30d4f':
weaved: Only build tests for Brillo eng builds.
Change-Id: I5117ca1a1169f072d0954020f1edae1433d51fc8
|
|
Avoid building weaved tests for targets that haven't set the
BRILLO variable.
Change-Id: I0b86c8f541d6854b057d31b7d8decb147e28ca44
|
|
am: 2e4acd4
* commit '2e4acd4f92c344c1da8cdce042615dac2ab28be5':
weaved: Implement new libweave's methods for WebServer interface
Change-Id: Ibec4752e2c7f9005be5df154841a751b6e0d9607
|
|
am: 3c0af52
* commit '3c0af52620beaf50927a887aed73babfbb39da66':
weaved: Implement new libweave's methods for WebServer interface
Change-Id: Ia7e8d212839217277a2e9bb1db93be0c6d0f58b7
|
|
The new version of libweave requires two more interface methods for
the web server - to remove previously registered HTTP(S) handlers.
Adding these methods...
Change-Id: I1a52fe8efab6affbf7b6e644b23453c14cc35e6d
|
|
am: 6a269a3
* commit '6a269a316d9f146d2f1381e816bda6284b23ebc4':
weaved: Fix OnProtocolHandlerConnected notification
|
|
Web server may have multiple protocol handlers on many ports, but
weave cares only for the default ones on port 80 and 443. When both
handlers are available, we invoke |server_available_callback_| callback.
The current code had a problem that https port was used in condition
twice and also when another protocol handler is connected after the
default http/https ones, the callback will be invoked multiple times.
BUG:27655609
Change-Id: I0982bb37d29974bf7de127405c2aca0190a48a3d
|
|
am: aa436b1
* commit 'aa436b16e8d3425302eed56ffb92ecef4bc8e909':
Port weaved to binder webservd interface
|
|
Change-Id: I94085e4cd6acbbf604fec221f35f84878ecfd778
Test: Can successfully connect to webservd
Bug: 27204884
|
|
am: 64ee4ff6a8
* commit '64ee4ff6a863ced08c98e36dd05a93221917c0a0':
Remove clang warning in unit tests
|
|
am: a0ff2eccb3
* commit 'a0ff2eccb34362a7c336284599b5bbbbebf49398':
Remove clang warning in unit tests
|
|
The new clang compiler compains about |state| and |origin| possibly
being uninitialized if StringToEnum fails. There is no harm of
initializing them to some default values...
BUG: 27437721
Change-Id: If8062a337d5530223fcab6480bbbeab257893d08
|
|
am: 7be8a4e415
* commit '7be8a4e415a0580fb2fbe3030facacd91a4c91c4':
Implement provider::Wifi::GetConnectedSsid()
|
|
am: 7fec91c0b6
* commit '7fec91c0b6b4cf8ba2926d6ba1a42f75aa4a8e4f':
Return calculated values in IsWifi*Supported
|
|
am: 5645f15604
* commit '5645f15604d06e3bc5663c272843a2465b08a8a6':
Remove connectivity_state_ member
|
|
am: 20f03a2966
* commit '20f03a2966652cf76ea2e7449cebaedb6b457acc':
Implement provider::Wifi::GetConnectedSsid()
|
|
am: 5db1247369
* commit '5db1247369bbf6abed9fbe0016e3a112a960bd4b':
Return calculated values in IsWifi*Supported
|
|
am: 026dce2bce
* commit '026dce2bce26ca52933ce10e200a196a6b59a63c':
Remove connectivity_state_ member
|
|
BUG: 25820726
Change-Id: Idc26aa3eb22aa40c7bf03827399e379db856578f
|
|
Also never downgrade is_*_supported_ when property changed, to avoid
inconsistencies with multiple adapters.
BUG: 27070625
Change-Id: I10bd6e13de4c26c57e1816832ad54c8326e4cab8
|
|
This value can be easily calculate from devices_.
Change-Id: I1da30d04991dd08f0b89600632ed9e818d9f6b2a
|
|
am: 240e401635
* commit '240e401635e226f903bffb234b9982acfdb74253':
weaved: Remove legacy command and state definitions
|
|
am: 0ee8bad158
* commit '0ee8bad1584f09092d9b86322d2f130b36fd73e1':
weaved: Re-enabled all compiler warnings
|