Commit 9df42e79 authored by khmel@chromium.org's avatar khmel@chromium.org Committed by Commit Bot

arc: Add extra logging for app mojom connection

TEST=Manually
BUG=b:117366533

Change-Id: I1038cbd9adf7e92df1953a428cd9b1a9bceb9b45
Reviewed-on: https://chromium-review.googlesource.com/c/1278291
Commit-Queue: Yury Khmel <khmel@chromium.org>
Commit-Queue: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: default avatarYusuke Sato <yusukes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599055}
parent f23bb6eb
...@@ -323,6 +323,7 @@ ArcAppListPrefs::ArcAppListPrefs( ...@@ -323,6 +323,7 @@ ArcAppListPrefs::ArcAppListPrefs(
prefs_(profile->GetPrefs()), prefs_(profile->GetPrefs()),
app_connection_holder_(app_connection_holder), app_connection_holder_(app_connection_holder),
weak_ptr_factory_(this) { weak_ptr_factory_(this) {
VLOG(1) << "ARC app list prefs created";
DCHECK(profile); DCHECK(profile);
DCHECK(app_connection_holder); DCHECK(app_connection_holder);
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI)); DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
...@@ -335,8 +336,10 @@ ArcAppListPrefs::ArcAppListPrefs( ...@@ -335,8 +336,10 @@ ArcAppListPrefs::ArcAppListPrefs(
base::Unretained(this))); base::Unretained(this)));
arc::ArcSessionManager* arc_session_manager = arc::ArcSessionManager::Get(); arc::ArcSessionManager* arc_session_manager = arc::ArcSessionManager::Get();
if (!arc_session_manager) if (!arc_session_manager) {
VLOG(1) << "ARC session manager is not available";
return; return;
}
DCHECK(arc::IsArcAllowedForProfile(profile)); DCHECK(arc::IsArcAllowedForProfile(profile));
...@@ -378,6 +381,8 @@ void ArcAppListPrefs::StartPrefs() { ...@@ -378,6 +381,8 @@ void ArcAppListPrefs::StartPrefs() {
arc_session_manager->AddObserver(this); arc_session_manager->AddObserver(this);
} }
VLOG(1) << "Registering host...";
app_connection_holder_->SetHost(this); app_connection_holder_->SetHost(this);
app_connection_holder_->AddObserver(this); app_connection_holder_->AddObserver(this);
if (!app_connection_holder_->IsConnected()) if (!app_connection_holder_->IsConnected())
...@@ -863,6 +868,8 @@ void ArcAppListPrefs::SetDefaultAppsFilterLevel() { ...@@ -863,6 +868,8 @@ void ArcAppListPrefs::SetDefaultAppsFilterLevel() {
} }
void ArcAppListPrefs::OnDefaultAppsReady() { void ArcAppListPrefs::OnDefaultAppsReady() {
VLOG(1) << "Default apps ready";
// Deprecated. Convert uninstalled packages info to hidden default apps and // Deprecated. Convert uninstalled packages info to hidden default apps and
// erase pending perf entry afterward. // erase pending perf entry afterward.
// TODO (khmel): Remove in M73 // TODO (khmel): Remove in M73
...@@ -960,6 +967,7 @@ void ArcAppListPrefs::SimulateDefaultAppAvailabilityTimeoutForTesting() { ...@@ -960,6 +967,7 @@ void ArcAppListPrefs::SimulateDefaultAppAvailabilityTimeoutForTesting() {
} }
void ArcAppListPrefs::OnConnectionReady() { void ArcAppListPrefs::OnConnectionReady() {
VLOG(1) << "App instance connection is ready.";
// Note, sync_service_ may be nullptr in testing. // Note, sync_service_ may be nullptr in testing.
sync_service_ = arc::ArcPackageSyncableService::Get(profile_); sync_service_ = arc::ArcPackageSyncableService::Get(profile_);
is_initialized_ = false; is_initialized_ = false;
...@@ -969,6 +977,7 @@ void ArcAppListPrefs::OnConnectionReady() { ...@@ -969,6 +977,7 @@ void ArcAppListPrefs::OnConnectionReady() {
} }
void ArcAppListPrefs::OnConnectionClosed() { void ArcAppListPrefs::OnConnectionClosed() {
VLOG(1) << "App instance connection is closed.";
DisableAllApps(); DisableAllApps();
installing_packages_count_ = 0; installing_packages_count_ = 0;
default_apps_installations_.clear(); default_apps_installations_.clear();
......
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