Reduce DBus ObjectUnknown ERROR to WARNING

Reduce DBus ObjectUnknown ERROR to WARNING

BUG=401571
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288446

[hashimoto, 2014-08-08 02:46:31]

https://codereview.chromium.org/455533002/diff/1/dbus/object_proxy.cc
File dbus/object_proxy.cc (right):

https://codereview.chromium.org/455533002/diff/1/dbus/object_proxy.cc#newcode573
dbus/object_proxy.cc:573: severity = logging::LOG_WARNING;
This variable looks unused and it seems we never reach this branch.

[hashimoto, 2014-08-08 07:28:46]

https://codereview.chromium.org/455533002/diff/1/dbus/object_proxy.cc
File dbus/object_proxy.cc (right):

https://codereview.chromium.org/455533002/diff/1/dbus/object_proxy.cc#newcode573
dbus/object_proxy.cc:573: severity = logging::LOG_WARNING;
On 2014/08/08 02:46:31, hashimoto wrote:
> This variable looks unused and it seems we never reach this branch.

Sorry, I dismissed "ignore_service_unknown_errors_ &&".
Please forget about the latter part.

[stevenjb, 2014-08-08 16:31:58]

ptal

https://codereview.chromium.org/455533002/diff/1/dbus/object_proxy.cc
File dbus/object_proxy.cc (right):

https://codereview.chromium.org/455533002/diff/1/dbus/object_proxy.cc#newcode573
dbus/object_proxy.cc:573: severity = logging::LOG_WARNING;
On 2014/08/08 07:28:46, hashimoto wrote:
> On 2014/08/08 02:46:31, hashimoto wrote:
> > This variable looks unused and it seems we never reach this branch.
> 
> Sorry, I dismissed "ignore_service_unknown_errors_ &&".
> Please forget about the latter part.

Oops, thanks for catching, fixed.

[hashimoto, 2014-08-08 17:17:42]

lgtm

[stevenjb, 2014-08-08 17:21:00]

The CQ bit was checked by stevenjb@chromium.org

[commit-bot: I haz the power, 2014-08-08 17:22:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/stevenjb@chromium.org/455533002/20001

[commit-bot: I haz the power, 2014-08-08 21:40:59]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_gn_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[commit-bot: I haz the power, 2014-08-08 21:57:23]

Change committed as 288446