[Mac] Don't offer profiles for billing addresses that have no address data....

[Mac] Don't offer profiles for billing addresses that have no address data.

BUG=http://crbug.com/50081
TEST=as in bug


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=54004

[Avi (use Gerrit), 2010-07-28 18:56:10]

This is what you had in mind? If so I'll do the other platforms too.

[James Hawkins, 2010-07-28 18:57:37]

On 2010/07/28 18:56:10, Avi wrote:
> This is what you had in mind? If so I'll do the other platforms too.

Yes, but you should use profile.IsEmpty().

[James Hawkins, 2010-07-28 18:57:50]

On 2010/07/28 18:57:37, James Hawkins wrote:
> On 2010/07/28 18:56:10, Avi wrote:
> > This is what you had in mind? If so I'll do the other platforms too.
> 
> Yes, but you should use profile.IsEmpty().

Sorry, I take that back.

[James Hawkins, 2010-07-28 18:58:35]

LGTM with comment.

http://codereview.chromium.org/2868076/diff/1/3
File chrome/browser/autofill/autofill_dialog_controller_mac.mm (right):

http://codereview.chromium.org/2868076/diff/1/3#newcode594
chrome/browser/autofill/autofill_dialog_controller_mac.mm:594: FieldTypeSet
fields;
Can you add a comment to clarify why we're making this check?

[Avi (use Gerrit), 2010-07-28 19:06:34]

> Can you add a comment to clarify why we're making this check?

Done; re-upped.

[James Hawkins, 2010-07-28 19:07:40]

SLGTM, thanks!