arc: Dont sync app list change caused by Arc opt out.

chrome/browser/ui/app_list/arc/arc_app_model_builder.cc

Index: chrome/browser/ui/app_list/arc/arc_app_model_builder.cc
diff --git a/chrome/browser/ui/app_list/arc/arc_app_model_builder.cc b/chrome/browser/ui/app_list/arc/arc_app_model_builder.cc
index b10e01bcb3e701829f7a7244bee98b8e5f449b63..ce621c923c6388c405d3aec12ab1ec0dded26987 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_model_builder.cc
+++ b/chrome/browser/ui/app_list/arc/arc_app_model_builder.cc
@@ -5,6 +5,7 @@
 #include "chrome/browser/ui/app_list/arc/arc_app_model_builder.h"
 
 #include "base/memory/ptr_util.h"
+#include "chrome/browser/chromeos/arc/arc_auth_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/app_list/arc/arc_app_item.h"
 
@@ -52,7 +53,9 @@ void ArcAppModelBuilder::OnAppRegistered(
 }
 
 void ArcAppModelBuilder::OnAppRemoved(const std::string& app_id) {
-  RemoveApp(app_id);
+  const arc::ArcAuthService* auth_service = arc::ArcAuthService::Get();
+  DCHECK(auth_service);
+  RemoveApp(app_id, !auth_service->IsArcEnabled());

[stevenjb, 2016/08/31 21:10:49]

This isn't a very intuitive value for 'local_change'. I would rename the
'local_change' parameter to 'unsynced_change' and add a comment here.

[khmel, 2016/09/01 00:03:02]

That is better, thanks

 }
 
 void ArcAppModelBuilder::OnAppIconUpdated(const std::string& app_id,