ARC Opt in notification not shown.

ARC Opt in notification not shown.

As the profile id is not set, the notifications are never shown to user.
Instead they land in the notification center, and user needs to click
on the "1" icon to see it.

This CL fixes the issue by setting a correct profile id on the
notification.

TEST=Tested manually on a fresh profile.
BUG=629399
Committed: https://crrev.com/3ae0dad4690b7579921cd503e6a863467ef27bfe
Cr-Commit-Position: refs/heads/master@{#406428}

[mtomasz, 2016-07-19 02:31:40]

Description was changed from

==========
ARC Opt in notification not shown.

As the profile id is not set, the notifications are never shown to user.
Instead they land in the notification center, and user needs to click
on the "1" icon to see it.

This CL fixes the issue by setting a correct profile id on the
notification.

TEST=Tested manually on a fresh profile.
BUG=629399
==========

to

==========
ARC Opt in notification not shown.

As the profile id is not set, the notifications are never shown to user.
Instead they land in the notification center, and user needs to click
on the "1" icon to see it.

This CL fixes the issue by setting a correct profile id on the
notification.

TEST=Tested manually on a fresh profile.
BUG=629399
==========

[mtomasz, 2016-07-19 02:31:40]

mtomasz@chromium.org changed reviewers:
+ oshima@chromium.org

[mtomasz, 2016-07-19 02:31:50]

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

[mtomasz, 2016-07-19 02:31:52]

@oshima: PTAL. Thanks!

[commit-bot: I haz the power, 2016-07-19 02:32:38]

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-07-19 03:11:07]

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

[commit-bot: I haz the power, 2016-07-19 03:11:07]

Dry run: This issue passed the CQ dry run.

[oshima, 2016-07-19 15:58:55]

lgtm

https://codereview.chromium.org/2160723004/diff/1/chrome/browser/chromeos/arc...
File chrome/browser/chromeos/arc/arc_auth_notification.cc (right):

https://codereview.chromium.org/2160723004/diff/1/chrome/browser/chromeos/arc...
chrome/browser/chromeos/arc/arc_auth_notification.cc:12: #include
"chrome/browser/profiles/profile.h"
I believe you don't need this.

[mtomasz, 2016-07-20 00:22:56]

https://codereview.chromium.org/2160723004/diff/1/chrome/browser/chromeos/arc...
File chrome/browser/chromeos/arc/arc_auth_notification.cc (right):

https://codereview.chromium.org/2160723004/diff/1/chrome/browser/chromeos/arc...
chrome/browser/chromeos/arc/arc_auth_notification.cc:12: #include
"chrome/browser/profiles/profile.h"
On 2016/07/19 15:58:55, oshima wrote:
> I believe you don't need this.

Done.

[mtomasz, 2016-07-20 00:23:01]

The CQ bit was checked by mtomasz@chromium.org

[commit-bot: I haz the power, 2016-07-20 00:23:36]

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-07-20 00:29:26]

Description was changed from

==========
ARC Opt in notification not shown.

As the profile id is not set, the notifications are never shown to user.
Instead they land in the notification center, and user needs to click
on the "1" icon to see it.

This CL fixes the issue by setting a correct profile id on the
notification.

TEST=Tested manually on a fresh profile.
BUG=629399
==========

to

==========
ARC Opt in notification not shown.

As the profile id is not set, the notifications are never shown to user.
Instead they land in the notification center, and user needs to click
on the "1" icon to see it.

This CL fixes the issue by setting a correct profile id on the
notification.

TEST=Tested manually on a fresh profile.
BUG=629399
==========

[commit-bot: I haz the power, 2016-07-20 00:29:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-20 00:31:25]

Description was changed from

==========
ARC Opt in notification not shown.

As the profile id is not set, the notifications are never shown to user.
Instead they land in the notification center, and user needs to click
on the "1" icon to see it.

This CL fixes the issue by setting a correct profile id on the
notification.

TEST=Tested manually on a fresh profile.
BUG=629399
==========

to

==========
ARC Opt in notification not shown.

As the profile id is not set, the notifications are never shown to user.
Instead they land in the notification center, and user needs to click
on the "1" icon to see it.

This CL fixes the issue by setting a correct profile id on the
notification.

TEST=Tested manually on a fresh profile.
BUG=629399

Committed: https://crrev.com/3ae0dad4690b7579921cd503e6a863467ef27bfe
Cr-Commit-Position: refs/heads/master@{#406428}
==========

[commit-bot: I haz the power, 2016-07-20 00:31:26]

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