Commit 330ebc0b authored by Shao-Chuan Lee's avatar Shao-Chuan Lee Committed by Commit Bot

Fix DCHECKs in HammerdClient D-Bus callbacks

DCHECKs in callbacks did not match corresponding signals.

Bug: 902717
Change-Id: I1fb9385ce0360445593c1c1b095706dfcd4821dc
Reviewed-on: https://chromium-review.googlesource.com/c/1331092Reviewed-by: default avatarRyo Hashimoto <hashimoto@chromium.org>
Commit-Queue: Shao-Chuan Lee <shaochuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#607531}
parent 59b93fee
......@@ -91,7 +91,7 @@ class HammerdClientImpl : public HammerdClient {
void OnBaseFirmwareUpdateStarted(dbus::Signal* signal) {
DCHECK_EQ(signal->GetInterface(), hammerd::kHammerdInterface);
DCHECK_EQ(signal->GetMember(), hammerd::kBaseFirmwareNeedUpdateSignal);
DCHECK_EQ(signal->GetMember(), hammerd::kBaseFirmwareUpdateStartedSignal);
for (auto& observer : observers_)
observer.BaseFirmwareUpdateStarted();
......@@ -99,7 +99,7 @@ class HammerdClientImpl : public HammerdClient {
void OnBaseFirmwareUpdateSucceeded(dbus::Signal* signal) {
DCHECK_EQ(signal->GetInterface(), hammerd::kHammerdInterface);
DCHECK_EQ(signal->GetMember(), hammerd::kBaseFirmwareNeedUpdateSignal);
DCHECK_EQ(signal->GetMember(), hammerd::kBaseFirmwareUpdateSucceededSignal);
for (auto& observer : observers_)
observer.BaseFirmwareUpdateSucceeded();
......@@ -107,7 +107,7 @@ class HammerdClientImpl : public HammerdClient {
void OnBaseFirmwareUpdateFailed(dbus::Signal* signal) {
DCHECK_EQ(signal->GetInterface(), hammerd::kHammerdInterface);
DCHECK_EQ(signal->GetMember(), hammerd::kBaseFirmwareNeedUpdateSignal);
DCHECK_EQ(signal->GetMember(), hammerd::kBaseFirmwareUpdateFailedSignal);
for (auto& observer : observers_)
observer.BaseFirmwareUpdateFailed();
......@@ -115,7 +115,7 @@ class HammerdClientImpl : public HammerdClient {
void OnPairChallengeSucceeded(dbus::Signal* signal) {
DCHECK_EQ(signal->GetInterface(), hammerd::kHammerdInterface);
DCHECK_EQ(signal->GetMember(), hammerd::kBaseFirmwareNeedUpdateSignal);
DCHECK_EQ(signal->GetMember(), hammerd::kPairChallengeSucceededSignal);
dbus::MessageReader reader(signal);
......@@ -132,7 +132,7 @@ class HammerdClientImpl : public HammerdClient {
void OnPairChallengeFailed(dbus::Signal* signal) {
DCHECK_EQ(signal->GetInterface(), hammerd::kHammerdInterface);
DCHECK_EQ(signal->GetMember(), hammerd::kBaseFirmwareNeedUpdateSignal);
DCHECK_EQ(signal->GetMember(), hammerd::kPairChallengeFailedSignal);
for (auto& observer : observers_)
observer.PairChallengeFailed();
......@@ -140,7 +140,7 @@ class HammerdClientImpl : public HammerdClient {
void OnInvalidBaseConnected(dbus::Signal* signal) {
DCHECK_EQ(signal->GetInterface(), hammerd::kHammerdInterface);
DCHECK_EQ(signal->GetMember(), hammerd::kBaseFirmwareNeedUpdateSignal);
DCHECK_EQ(signal->GetMember(), hammerd::kInvalidBaseConnectedSignal);
for (auto& observer : observers_)
observer.InvalidBaseConnected();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment