mac: Prevent Address Book permissions dialog from erroneously appearing.

chrome/browser/ui/autofill/chrome_autofill_client_mac.mm

+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/autofill/chrome_autofill_client.h"
+
+#import "chrome/browser/autofill/autofill_keystone_observer_mac.h"
+#include "components/autofill/core/browser/personal_data_manager.h"
+
+namespace autofill {
+
+void ChromeAutofillClient::AddKeystoneObserver() {
+  DCHECK(!keystone_observer_);
+  keystone_observer_ = new AutofillKeystoneObserverMac(this);
+}
+
+void ChromeAutofillClient::RemoveKeystoneObserver() {
+  DCHECK(keystone_observer_);
+  delete keystone_observer_;
+  keystone_observer_ = NULL;
+}
+
+void ChromeAutofillClient::OnKeystoneNotification(
+    keystone_glue::AutoupdateStatus status) {
+  switch (status) {
+    case keystone_glue::kAutoupdateInstalling:
+    case keystone_glue::kAutoupdateInstalled: {
+      PersonalDataManager* manager = GetPersonalDataManager();
+      if (manager)

[Ilya Sherman, 2014/06/14 01:18:51]

Why might the manager be null?

[erikchen, 2014/06/16 20:30:46]

There is no comment indicating that the method is guaranteed to return a
non-NULL pointer. Is this the case? If so, I will update the method and this
logic.

+        manager->BinaryChanging();
+      return;
+    }
+    default:
+      return;
+  }
+}
+
+}  // namespace autofill