mac: Don't show access Address Book prompt if user has any Autofill state.

mac: Two small changes to Access Addres Book prompt.

- Don't show access Address Book prompt if user has any Autofill state.
- Eliminate flickering of the access Address Book prompt.

BUG=414359
Committed: https://crrev.com/ca7fd5a44b5ce2ad379ec7dc54e14d0d00c83a8a
Cr-Commit-Position: refs/heads/master@{#295573}

[erikchen, 2014-09-18 00:45:51]

Patchset #1 (id:1) has been deleted

[erikchen, 2014-09-18 00:46:22]

erikchen@chromium.org changed reviewers:
+ estade@chromium.org

[erikchen, 2014-09-18 00:46:22]

erikchen@chromium.org changed reviewers:
+ estade@chromium.org

[erikchen, 2014-09-18 00:46:22]

estade: Please review.

[erikchen, 2014-09-18 00:46:23]

estade: Please review.

[erikchen, 2014-09-18 00:47:39]

Oops, sorry for the double email. rietvald fail. It turned my new lines into
email sends. :(

I wanted to point out that the prompt is still disabled, since I haven't
implemented the max # of times to show prompt logic.

[Evan Stade, 2014-09-18 01:17:53]

lgtm, but can you add a test?

[erikchen, 2014-09-18 18:09:45]

added test.

[erikchen, 2014-09-18 18:09:49]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2014-09-18 18:10:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/575283003/40001

[Evan Stade, 2014-09-18 18:30:54]

On 2014/09/18 18:10:38, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/575283003/40001

thanks, test lgtm

[commit-bot: I haz the power, 2014-09-18 19:37:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-18 19:37:55]

Try jobs failed on following builders:
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[erikchen, 2014-09-18 21:02:54]

estade: Fixed another test. Fixed some broken (unused) logic in that test class.

https://codereview.chromium.org/575283003/diff/60001/components/autofill/core...
File components/autofill/core/browser/autofill_manager_unittest.cc (left):

https://codereview.chromium.org/575283003/diff/60001/components/autofill/core...
components/autofill/core/browser/autofill_manager_unittest.cc:98:
std::remove(credit_cards_.begin(), credit_cards_.end(), credit_card),
This logic is incorrect. std::remove and erase() both remove elements from the
vector.

[erikchen, 2014-09-18 21:02:59]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2014-09-18 21:04:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/575283003/60001

[Evan Stade, 2014-09-18 21:08:36]

On 2014/09/18 21:04:22, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/575283003/60001

lgtm again

[commit-bot: I haz the power, 2014-09-18 21:59:40]

Committed patchset #3 (id:60001) as 6d4b0b52b9492819268c145130ec863677b4cea6

[commit-bot: I haz the power, 2014-09-18 22:00:41]

Patchset 3 (id:??) landed as
https://crrev.com/ca7fd5a44b5ce2ad379ec7dc54e14d0d00c83a8a
Cr-Commit-Position: refs/heads/master@{#295573}