Disable kDesktopNotificationPosition for ChromeOS

It used to be possible to change the position of the notification bubble on ChromeOS. Now that the wrench menu for doing so is disabled it is no longer possible to change the balloon position.
Force the pref to be the default position such that users are not stuck with the notification bubble in a weird place.

Bug=None
Test=Manual

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

[Elliot Glaysher, 2012-05-01 17:38:57]

I don't think I'm qualified to review this.

[sky, 2012-05-01 17:55:48]

I don't know this code either. Is there someone else you can get to review?

[stevenjb, 2012-05-02 23:07:49]

Is there an issue related to this?

I'm not sure that I follow step 1), but the wrench menu has been removed from
notifications on ChromeOS (which are also in the process of being replace).

[pkotwicz, 2012-05-03 01:03:50]

I saw that the wrench menu has been removed from notifications. I still get the
animation in the wrong place on the screen.
I don't mind deleting this CL if we are replacing the notifications as well.

[pkotwicz, 2012-05-03 01:04:18]

The balloon is on the top right for me because it has been saved in my settings
as such

[stevenjb, 2012-05-03 18:02:26]

On 2012/05/03 01:04:18, pkotwicz wrote:
> The balloon is on the top right for me because it has been saved in my
settings
> as such

Ah, that's a problem. Can you instead change the behavior to ignore the setting
on ChromeOS?

[pkotwicz, 2012-05-05 02:54:47]

Change as requested.

[stevenjb, 2012-05-08 16:57:44]

lgtm

http://codereview.chromium.org/10260021/diff/13001/chrome/browser/notificatio...
File chrome/browser/notifications/notification_ui_manager_impl.cc (right):

http://codereview.chromium.org/10260021/diff/13001/chrome/browser/notificatio...
chrome/browser/notifications/notification_ui_manager_impl.cc:60: #elif
defined(OS_MACOSX)
nit: These should be two separate #if clauses, since they are not related.

[pkotwicz, 2012-05-08 17:41:33]

Sky for rubberstamp

[sky, 2012-05-08 17:49:05]

http://codereview.chromium.org/10260021/diff/13001/chrome/browser/notificatio...
File chrome/browser/notifications/notification_ui_manager_impl.cc (right):

http://codereview.chromium.org/10260021/diff/13001/chrome/browser/notificatio...
chrome/browser/notifications/notification_ui_manager_impl.cc:59:
BalloonCollection::DEFAULT_POSITION);
Should this be in notification_prefs_manager.cc  ?

[pkotwicz, 2012-05-09 05:39:43]

Change as requested

[sky, 2012-05-09 14:26:03]

LGTM

[commit-bot: I haz the power, 2012-05-09 15:06:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/10260021/13002

[commit-bot: I haz the power, 2012-05-09 16:23:32]

Change committed as 136043