Commit 808dea13 authored by Azeem Arshad's avatar Azeem Arshad Committed by Commit Bot

[AndroidSms] Add logging for pairing state tracking.

This CL adds additional logging to help understand what is triggering
pairing state change in https://crbug.com/1060823

Bug: 1060823
Change-Id: I774167ae0e420bae11c0c58a9caf8937f0c93961
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2138562
Commit-Queue: Azeem Arshad <azeemarshad@chromium.org>
Reviewed-by: default avatarKyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#756915}
parent e0462d08
......@@ -69,6 +69,7 @@ void AndroidSmsPairingStateTrackerImpl::OnCookiesRetrieved(
}
}
PA_LOG(INFO) << "No Pairing cookie found";
was_paired_on_last_update_ = false;
if (was_previously_paired != was_paired_on_last_update_)
NotifyPairingStateChanged();
......@@ -88,6 +89,8 @@ void AndroidSmsPairingStateTrackerImpl::OnCookieChange(
void AndroidSmsPairingStateTrackerImpl::OnInstalledAppUrlChanged() {
// If the app URL changed, stop any ongoing cookie monitoring and attempt to
// add a new change listener.
PA_LOG(INFO) << "Installed app url changed to " << GetPairingUrl()
<< ". Updating cookie listeners.";
cookie_listener_receiver_.reset();
AddCookieChangeListener();
}
......
......@@ -81,12 +81,15 @@ bool AndroidSmsAppInstallingStatusObserver::
void AndroidSmsAppInstallingStatusObserver::UpdatePwaInstallationState() {
if (!DoesFeatureStateAllowInstallation()) {
PA_LOG(INFO)
<< "Feature state does not allow installation, tearing down App.";
// The feature is disabled, ensure that the integration cookie is removed.
android_sms_app_helper_delegate_->TearDownAndroidSmsApp();
return;
}
if (android_sms_app_helper_delegate_->HasAppBeenManuallyUninstalledByUser()) {
PA_LOG(INFO) << "App was manually uninstalled by user, tearing down App.";
feature_state_manager_->SetFeatureEnabledState(mojom::Feature::kMessages,
false);
......
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