Update test code to check what is displayed in chooser's table view

Update test code to check what is displayed in chooser's table view

This CL added more test code to ChooserDialogCocoaControllerTest and
ChooserContentViewTest to check what is displayed in chooser's table
view.

BUG=638350
Committed: https://crrev.com/e1bc6e5a4bcdca1ab5a242c504a3a65355ce970c
Cr-Commit-Position: refs/heads/master@{#412415}

[juncai, 2016-08-16 21:11:06]

The CQ bit was checked by juncai@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-16 21:11:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-16 21:49:17]

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

[commit-bot: I haz the power, 2016-08-16 21:49:18]

Dry run: This issue passed the CQ dry run.

[juncai, 2016-08-16 21:56:20]

juncai@chromium.org changed reviewers:
+ msw@chromium.org, rsesek@chromium.org

[juncai, 2016-08-16 21:56:20]

msw@chromium.org: Please review changes in 
//chrome/browser/ui/views/chooser_content_view_unittest.cc

rsesek@chromium.org: Please review changes in 
//chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa_controller_unittest.mm

[Robert Sesek, 2016-08-16 22:30:17]

https://codereview.chromium.org/2256453002/diff/1/chrome/browser/ui/cocoa/ext...
File
chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa_controller_unittest.mm
(right):

https://codereview.chromium.org/2256453002/diff/1/chrome/browser/ui/cocoa/ext...
chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa_controller_unittest.mm:94:
EXPECT_EQ(
How about:
 
EXPECT_NSEQ(l10n_util::GetStringNSString(IDS_DEVICE_CHOOSER_NO_DEVICES_FOUND_PROMPT),
            [[table_view_ preparedCellAtColumn:0 row:0] stringValue]);

https://codereview.chromium.org/2256453002/diff/1/chrome/browser/ui/cocoa/ext...
chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa_controller_unittest.mm:117:
EXPECT_EQ(base::ASCIIToUTF16("a"),
Similarly,

EXPECT_NSEQ(@"a", …);

[juncai, 2016-08-16 23:01:48]

The CQ bit was checked by juncai@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-16 23:02:30]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[juncai, 2016-08-16 23:03:21]

https://codereview.chromium.org/2256453002/diff/1/chrome/browser/ui/cocoa/ext...
File
chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa_controller_unittest.mm
(right):

https://codereview.chromium.org/2256453002/diff/1/chrome/browser/ui/cocoa/ext...
chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa_controller_unittest.mm:94:
EXPECT_EQ(
On 2016/08/16 22:30:17, Robert Sesek wrote:
> How about:
>  
>
EXPECT_NSEQ(l10n_util::GetStringNSString(IDS_DEVICE_CHOOSER_NO_DEVICES_FOUND_PROMPT),
>             [[table_view_ preparedCellAtColumn:0 row:0] stringValue]);

Done.

https://codereview.chromium.org/2256453002/diff/1/chrome/browser/ui/cocoa/ext...
chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa_controller_unittest.mm:117:
EXPECT_EQ(base::ASCIIToUTF16("a"),
On 2016/08/16 22:30:17, Robert Sesek wrote:
> Similarly,
> 
> EXPECT_NSEQ(@"a", …);

Done.

[Robert Sesek, 2016-08-16 23:16:23]

lgtm

[commit-bot: I haz the power, 2016-08-17 00:51:06]

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

[commit-bot: I haz the power, 2016-08-17 00:51:07]

Dry run: This issue passed the CQ dry run.

[msw, 2016-08-17 01:11:05]

lgtm

[juncai, 2016-08-17 01:33:58]

The CQ bit was checked by juncai@chromium.org

[commit-bot: I haz the power, 2016-08-17 01:34:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-17 01:38:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-17 01:40:43]

Description was changed from

==========
Update test code to check what is displayed in chooser's table view

This CL added more test code to ChooserDialogCocoaControllerTest and
ChooserContentViewTest to check what is displayed in chooser's table
view.

BUG=638350
==========

to

==========
Update test code to check what is displayed in chooser's table view

This CL added more test code to ChooserDialogCocoaControllerTest and
ChooserContentViewTest to check what is displayed in chooser's table
view.

BUG=638350

Committed: https://crrev.com/e1bc6e5a4bcdca1ab5a242c504a3a65355ce970c
Cr-Commit-Position: refs/heads/master@{#412415}
==========

[commit-bot: I haz the power, 2016-08-17 01:40:44]

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