| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
This is the new name. It has the same meaning but additionally with a "you can use the headers" permission.
Rename direct_dependent_configs to public_configs. This is the new name with identical meaning.
TBR=jamesr
Review URL: https://codereview.chromium.org/595073002
Cr-Commit-Position: refs/heads/master@{#296302}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, ObjectProxy::CallMethodAndBlock() returns NULL on
failure but the D-Bus error details are not passed to the caller.
Added a method that would return the D-Bus error information
to the caller and switched ObjectProxy::CallMethodAndBlock()
to use the new method instead.
BUG=chromium:414838
TEST=Build Chrome
TEST=ninja -C out/Debug dbus_unittests && out/Debug/dbus_unittests
Review URL: https://codereview.chromium.org/563763004
Cr-Commit-Position: refs/heads/master@{#295212}
|
|
|
|
|
|
|
|
|
|
|
| |
This CL introduces util_unittest.cc, which acts as a place for current and
future functions in util.h.
BUG=411108
Review URL: https://codereview.chromium.org/552433002
Cr-Commit-Position: refs/heads/master@{#293793}
|
|
|
|
|
|
|
|
|
| |
BUG=
R=hclam@chromium.org
Review URL: https://codereview.chromium.org/547793002
Cr-Commit-Position: refs/heads/master@{#293774}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There is a race condition in the way that match rules get set up for object
proxies created in response to GetManagedObjects that may cause us the miss
PropertiesChanged signals if they're received before the match rule and filter
function get added by ObjectProxy.
This patch changes this to work the "intended" way: ObjectManager now adds a
single match rule for its corresponding service name, and specifically for the
org.freedesktop.DBus.Properties.PropertiesChanged signal. Once it receives the
signal, ObjectManager dispatches the signal to the corresponding PropertySet.
BUG=407109,400768
TEST=dbus_unittests
Review URL: https://codereview.chromium.org/510863002
Cr-Commit-Position: refs/heads/master@{#293551}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also, remove the deprecated base::MessageLoop::Run() calls
everywhere in dbus unittests.
BUG=365669
R=armansito@chromium.org
Review URL: https://codereview.chromium.org/523623003
Cr-Commit-Position: refs/heads/master@{#293113}
|
|
|
|
|
|
|
|
|
|
| |
TBR=haruki
BUG=407063
Review URL: https://codereview.chromium.org/501953002
Cr-Commit-Position: refs/heads/master@{#291664}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
dbus::Message::GetSignature() was incorrectly renamed to
dbus::Message::GetDataSigature() in
https://codereview.chromium.org/502793002. This CL undoes that.
BUG=None
TEST=dbus_unittests
Review URL: https://codereview.chromium.org/494263003
Cr-Commit-Position: refs/heads/master@{#291660}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The new MessageReader::GetSignature() method returns the DBus signature
of the value at the current iterator position, which is needed for
implementing automatic de-serialization of complex DBus messages.
BUG=None
TEST=dbus_unittests
Review URL: https://codereview.chromium.org/502793002
Cr-Commit-Position: refs/heads/master@{#291643}
|
|
|
|
|
|
|
|
|
| |
BUG=401571
Review URL: https://codereview.chromium.org/455533002
Cr-Commit-Position: refs/heads/master@{#288446}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288446 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Since dbus is only used on linux, don't bother with windows specific symbol exporting.
BUG=398644
Review URL: https://codereview.chromium.org/429043003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288034 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This previous CL added attributes to some template specializations:
https://codereview.chromium.org/408143012
This breaks gcc 4.8, which gives the following warning:
../../dbus/property.h:412:62: error: type attributes ignored after type is already defined [-Werror=attributes]
Since the template class already has the attribute, I suspect that the
additional attribute specifications are redundant. ie I think we can
remove them.
Review URL: https://codereview.chromium.org/423233003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286567 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
dbus::Property<std::vector<uint8> > isn't instantiated in libdbus in non-ChromeOS build + latest clang.
That causes instantiation of the template in dbus_unittests, and causes link failure.
Review URL: https://codereview.chromium.org/408143012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285475 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In GN it is redundat to include the parent directory name, as in
ui/gfx:gfx_test_support. Instead we can write it as ui/gfx:test_support
which is shorter and nicer to read.
Targets found with the following command line:
$ for f in $(g ls-files | grep BUILD); do grep _test_support $f; echo
$f; done
BUG=None
TEST=None
R=brettw@chromium.org
TBR=darin@chromium.org
Review URL: https://codereview.chromium.org/409493002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284800 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=393266
For log only change to profile_io_data.cc (reviewed by mattm)
TBR=mmenke@chromium.org
Review URL: https://codereview.chromium.org/385993004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283648 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(https://codereview.chromium.org/354553002/)
Reason for revert:
This causes link error:
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Builder%20%28dbg%29/builds/48613/steps/compile/logs/stdio
Original issue's description:
> [DBus] Explicitly instantiate dbus::Property variants
>
> dbus::Property<std::vector<uint8> > isn't instantiated in libdbus in non-ChromeOS build + latest clang.
> That causes instantiation of the template in dbus_unittests, and causes link failure.
>
> BUG=
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=280264
TBR=keybuk@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=
Review URL: https://codereview.chromium.org/353063007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280267 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
dbus::Property<std::vector<uint8> > isn't instantiated in libdbus in non-ChromeOS build + latest clang.
That causes instantiation of the template in dbus_unittests, and causes link failure.
BUG=
Review URL: https://codereview.chromium.org/354553002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280264 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Still not hooked up and not quite everything compiles but this should be the bulk of the functionality.
BUG=
R=scottmg@chromium.org
Review URL: https://codereview.chromium.org/328313003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276774 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=322664
Review URL: https://codereview.chromium.org/307423005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275563 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None
R=scottmg@chromium.org
TBR=satorux@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/301233007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@274491 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added functions AppendValueData and AppendValueDataAsVariant to values_util.h.
These functions are useful for writing both basic types and variant lists and
dictionaries to a message writer. Lists and dictionaries are written with type
"av" and "a{sv}", respectively.
BUG=359413
TEST=dbus_unittests,chromeos_unittests
Review URL: https://codereview.chromium.org/221393004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272167 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=sky@chromium.org
Review URL: https://codereview.chromium.org/270183002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@268754 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Move IsStringASCII/UTF8 to base namespace.
>
> Use StringPiece for IsStringUTF8.
>
> TBR=sky
>
> Review URL: https://codereview.chromium.org/196793010
TBR=brettw@chromium.org
Review URL: https://codereview.chromium.org/198163004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@257533 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Use StringPiece for IsStringUTF8.
TBR=sky
Review URL: https://codereview.chromium.org/196793010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@257524 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=351229
TEST=dbus_unittests
Review URL: https://codereview.chromium.org/199573003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@257315 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since the ObjectManager class managers the set of object proxies and
properties structures for remote service it needs to handle the
NameOwnerChanged signal, deleting the current set of objects when a
previously owned name changes to a new owner (or is released) and
re-requesting the set of managed objects when a name is claimed by
a new owner.
BUG=346975
TEST=dbus_unittests
Review URL: https://codereview.chromium.org/177703006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255495 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make memory returned by MessageReader::PopArrayOfBytes()
const to make it clearer that ownership remains with the
MessageReader.
Also update PopArrayOfStrings() and PopArrayOfObjectPaths()
to clear the passed-in vectors before appending to them.
BUG=none
TBR=isherman@chromium.org,mvanouwerkerk@chromium.org
Review URL: https://codereview.chromium.org/176693003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252922 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Along the way, fix the order in the test service used in unit tests.
BUG=332120
TEST=dbus_unittests pass as before
Review URL: https://codereview.chromium.org/125673003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@243770 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=327172
Review URL: https://codereview.chromium.org/115173002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@241222 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Also fix a bug that prevented that from actually working.
BUG=236893
R=stevenjb@chromium.org
Review URL: https://codereview.chromium.org/108563003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@239269 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
base namespace.
BUG=
R=viettrungluu@chromium.org
Review URL: https://codereview.chromium.org/102843002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238465 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To prevent Shill Service DBus ObjectProxy instances from accumulating,
remove them when the service becomes inactive.
Hopefully includes ASAN fixes.
BUG=223483
TBR=hashimoto@chromium.org
Review URL: https://codereview.chromium.org/26289002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227371 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Track active references in ShillClientHelper (Take 2)
> To prevent Shill Service DBus ObjectProxy instances from accumulating,
> remove them when the service becomes inactive.
>
> Original CL: https://codereview.chromium.org/23658053/
>
> BUG=223483
> TBR=hashimoto@chromium.org
>
> Review URL: https://codereview.chromium.org/24558004
Failure:
http://build.chromium.org/p/chromium.memory/builders/Chromium%20OS%20%28amd64%29%20ASAN/builds/6931
TBR=stevenjb@chromium.org
Review URL: https://codereview.chromium.org/26075002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227139 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To prevent Shill Service DBus ObjectProxy instances from accumulating,
remove them when the service becomes inactive.
Original CL: https://codereview.chromium.org/23658053/
BUG=223483
TBR=hashimoto@chromium.org
Review URL: https://codereview.chromium.org/24558004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227100 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Split ConnectToNameOwnerChangedSignal() from ConnectToSignalInternal().
Add WaitForServiceToBeAvailable()
Add test.
BUG=141009
TEST=dbus_unittests
R=satorux@chromium.org
Review URL: https://codereview.chromium.org/25488002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226428 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
thread
Accessing the callback on the D-Bus thread while changing its value may result in a race condition.
Change the type of SetNameOwnerChangedCallback's argument from Signal to strings to stop worrying about on which thread the Signal gets released.
BUG=298747
TEST=dbus_unittests
R=satorux@chromium.org
Review URL: https://codereview.chromium.org/24673006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225675 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=dbus_unittests
R=satorux@chromium.org
Review URL: https://codereview.chromium.org/24557002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225362 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By exposing TaskRunners,
we'll be able to use utility functions like base::PostTaskAndReplyWithResult
BUG=None
TEST=git cl try
R=satorux@chromium.org
Review URL: https://codereview.chromium.org/24554002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225324 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use after free on ASAN chromiumos.
> Track active references in ShillClientHelper
> To prevent Shill Service DBus ObjectProxy instances from accumulating,
> remove them when the service becomes inactive.
>
> BUG=223483
>
> Review URL: https://chromiumcodereview.appspot.com/23658053
TBR=stevenjb@chromium.org
Review URL: https://codereview.chromium.org/24293002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224204 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
To prevent Shill Service DBus ObjectProxy instances from accumulating,
remove them when the service becomes inactive.
BUG=223483
Review URL: https://chromiumcodereview.appspot.com/23658053
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224179 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The NameHasNoOwner errors are emitted as Chrome tries to get the
identity of the service when connecting to signals for sender
verification (crbug.com/140938), but the service may not yet be
running, which is all right, hence the NameHasNoOwner errors here
should be just supressed.
BUG=273829
TEST=out/Debug/dbus_unittests --gtest_filter=SignalSenderVerificationTest.TestSignalAccepted; confirm that NameHasNoOwner is no longre emitted. Previously, an error message like below was emited:
[12721:12722:0816/144031:3991948732792:ERROR:bus.cc(892)] Failed to get name owner. Got org.freedesktop.DBus.Error.NameHasNoOwner: Could not get owner of name 'org.chromium.TestService': no such name
Review URL: https://chromiumcodereview.appspot.com/23143004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@217980 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We've seen some cases in tests where a Chromium process winds up in a
temporarily unkillable state, causing the dbus-daemon to believe that
it still actively owns org.chromium.LibCrosService. This makes
attempts to restart the UI fail, as the browser dies when it cannot
take ownership of this service name. The reason it can't is because
Chromium currently doesn't allow other processes to steal ownership --
and the unkillable process is holding onto the token.
This can be remedied by providing certain options when ownership of
the service name is taken, options that allow other processes to seize
ownership if they so choose. The ramifications of this are discussed
further in the bug.
BUG=chromium:261381
TEST=new unit test in dbus_unittest
TEST=run the following as chronos on a device: "gdbus call --system --dest org.freedesktop.DBus --object-path /org/freedesktop/DBus --method org.freedesktop.DBus.RequestName org.chromium.LibCrosService 7"
TEST=This should return (uint32 1,)
Review URL: https://chromiumcodereview.appspot.com/20555003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214589 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
No behavior change.
BUG=263960
R=satorux@chromium.org
Review URL: https://codereview.chromium.org/21131003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214521 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
extensions/, google_apis/, gpu/, ipc/, jingle/.
BUG=260807
TEST=none
TBR=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/19607005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212157 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=
TEST=
Review URL: https://chromiumcodereview.appspot.com/18083039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211070 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
google_apis/, gpu/, ipc/, jingle/.
BUG=254986
TEST=none
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/18156002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@209149 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=247723
TEST=everything compiles OK
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/17390010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207507 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
write dbus:: everywhere. Remove some other dbus:: usages in the dbus namespace.
Review URL: https://chromiumcodereview.appspot.com/16012018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206010 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=247723
TEST=none
TBR=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/16123026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205457 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
the new service owner.
The mtpd dbus service may not start right away. Any attempts to use it may be racy due to the lack of a service owner. Listening for service owner changes fixes this race.
BUG=241302
TEST=Manual, see bug for repro case.
Review URL: https://chromiumcodereview.appspot.com/15741025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205331 0039d316-1c4b-4281-b951-d872f2087c98
|