Implement update notifications for ChromeVox Next.

Implement update notifications for ChromeVox Next.

- adds a notification on startup alerting the user to a new ChromeVox experience
- adds a skeleton page to be filled with content for the transition intro/tutorial
- fixes the alert output rule to ignore descendants in the output if the alert has a name
- adds a new command to open the "new" or "next" announcements/release notes/tutorial
- fixes two issues with the notification center:
1. the focus wasn't being placed on the first message view item
2. the scroll view parenting the message views was receiving focus but has no actionable functionality from the keyboard. Removed it from being focusable.
The outcome is when hitting alt+shift+n, an actionable button gets focused with or wihtout a notification being present (do not disturb gets focus if there are no messages).
Previously, alt+shift+n focused the scrollview, which has no name/role, so gave no meaningful feedback.
BUG=618094
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/9e0f3f0f765bc0625c7988086d8e9ffefd97d337
Cr-Commit-Position: refs/heads/master@{#399051}

[David Tseng, 2016-06-08 18:55:16]

Description was changed from

==========
Implement update notifications for ChromeVox Next.
==========

to

==========
Implement update notifications for ChromeVox Next.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-06-08 19:06:27]

Description was changed from

==========
Implement update notifications for ChromeVox Next.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Implement update notifications for ChromeVox Next.

- adds a notification on startup alerting the user to a new ChromeVox experience
- adds a skeleton page to be filled with content for the transition
intro/tutorial
- fixes the alert output rule to ignore descendants in the output if the alert
has a name
- adds a new command to open the "new" or "next" announcements/release
notes/tutorial
- fixes two issues with the notification center:
1. the focus wasn't being placed on the first message view item
2. the scroll view parenting the message views was receiving focus but has no
actionable functionality from the keyboard. Removed it from being focusable.
The outcome is when hitting alt+shift+n, an actionable button gets focused with
or wihtout a notification being present (do not disturb gets focus if there are
no messages).
Previously, alt+shift+n focused the scrollview, which has no name/role, so gave
no meaningful feedback.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-06-08 19:06:27]

dtseng@chromium.org changed reviewers:
+ dmazzoni@chromium.org, yoshiki@chromium.org

[David Tseng, 2016-06-08 19:07:17]

dmazzoni: entire cl
yoshiki: message center related files.

[David Tseng, 2016-06-08 20:04:49]

Description was changed from

==========
Implement update notifications for ChromeVox Next.

- adds a notification on startup alerting the user to a new ChromeVox experience
- adds a skeleton page to be filled with content for the transition
intro/tutorial
- fixes the alert output rule to ignore descendants in the output if the alert
has a name
- adds a new command to open the "new" or "next" announcements/release
notes/tutorial
- fixes two issues with the notification center:
1. the focus wasn't being placed on the first message view item
2. the scroll view parenting the message views was receiving focus but has no
actionable functionality from the keyboard. Removed it from being focusable.
The outcome is when hitting alt+shift+n, an actionable button gets focused with
or wihtout a notification being present (do not disturb gets focus if there are
no messages).
Previously, alt+shift+n focused the scrollview, which has no name/role, so gave
no meaningful feedback.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Implement update notifications for ChromeVox Next.

- adds a notification on startup alerting the user to a new ChromeVox experience
- adds a skeleton page to be filled with content for the transition
intro/tutorial
- fixes the alert output rule to ignore descendants in the output if the alert
has a name
- adds a new command to open the "new" or "next" announcements/release
notes/tutorial
- fixes two issues with the notification center:
1. the focus wasn't being placed on the first message view item
2. the scroll view parenting the message views was receiving focus but has no
actionable functionality from the keyboard. Removed it from being focusable.
The outcome is when hitting alt+shift+n, an actionable button gets focused with
or wihtout a notification being present (do not disturb gets focus if there are
no messages).
Previously, alt+shift+n focused the scrollview, which has no name/role, so gave
no meaningful feedback.
BUG=618094

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[dmazzoni, 2016-06-09 15:45:02]

lgtm but I don't know the ui/message_center code

https://codereview.chromium.org/2054473003/diff/40001/chrome/browser/resource...
File chrome/browser/resources/chromeos/chromevox/common/command_store.js
(right):

https://codereview.chromium.org/2054473003/diff/40001/chrome/browser/resource...
chrome/browser/resources/chromeos/chromevox/common/command_store.js:158:
'showNextUpdatePage': {announce: false},
Add a category for this so it shows up in the ChromeVox menus

https://codereview.chromium.org/2054473003/diff/40001/ui/message_center/views...
File ui/message_center/views/message_center_bubble.cc (right):

https://codereview.chromium.org/2054473003/diff/40001/ui/message_center/views...
ui/message_center/views/message_center_bubble.cc:108:
->GetNextFocusableView(NULL, NULL, false, false)
nit: nullptr

[yoshiki, 2016-06-09 17:43:39]

the message center code lgtm

[David Tseng, 2016-06-09 18:47:00]

https://codereview.chromium.org/2054473003/diff/40001/chrome/browser/resource...
File chrome/browser/resources/chromeos/chromevox/common/command_store.js
(right):

https://codereview.chromium.org/2054473003/diff/40001/chrome/browser/resource...
chrome/browser/resources/chromeos/chromevox/common/command_store.js:158:
'showNextUpdatePage': {announce: false},
On 2016/06/09 15:45:02, dmazzoni wrote:
> Add a category for this so it shows up in the ChromeVox menus

Done.

https://codereview.chromium.org/2054473003/diff/40001/ui/message_center/views...
File ui/message_center/views/message_center_bubble.cc (right):

https://codereview.chromium.org/2054473003/diff/40001/ui/message_center/views...
ui/message_center/views/message_center_bubble.cc:108:
->GetNextFocusableView(NULL, NULL, false, false)
On 2016/06/09 15:45:02, dmazzoni wrote:
> nit: nullptr

Done.

[David Tseng, 2016-06-09 21:34:24]

The CQ bit was checked by dtseng@chromium.org

[David Tseng, 2016-06-09 21:34:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
yoshiki@chromium.org, dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2054473003/#ps80001
(title: "Fix test.")

[commit-bot: I haz the power, 2016-06-09 21:34:59]

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

[commit-bot: I haz the power, 2016-06-09 23:44:12]

Description was changed from

==========
Implement update notifications for ChromeVox Next.

- adds a notification on startup alerting the user to a new ChromeVox experience
- adds a skeleton page to be filled with content for the transition
intro/tutorial
- fixes the alert output rule to ignore descendants in the output if the alert
has a name
- adds a new command to open the "new" or "next" announcements/release
notes/tutorial
- fixes two issues with the notification center:
1. the focus wasn't being placed on the first message view item
2. the scroll view parenting the message views was receiving focus but has no
actionable functionality from the keyboard. Removed it from being focusable.
The outcome is when hitting alt+shift+n, an actionable button gets focused with
or wihtout a notification being present (do not disturb gets focus if there are
no messages).
Previously, alt+shift+n focused the scrollview, which has no name/role, so gave
no meaningful feedback.
BUG=618094

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Implement update notifications for ChromeVox Next.

- adds a notification on startup alerting the user to a new ChromeVox experience
- adds a skeleton page to be filled with content for the transition
intro/tutorial
- fixes the alert output rule to ignore descendants in the output if the alert
has a name
- adds a new command to open the "new" or "next" announcements/release
notes/tutorial
- fixes two issues with the notification center:
1. the focus wasn't being placed on the first message view item
2. the scroll view parenting the message views was receiving focus but has no
actionable functionality from the keyboard. Removed it from being focusable.
The outcome is when hitting alt+shift+n, an actionable button gets focused with
or wihtout a notification being present (do not disturb gets focus if there are
no messages).
Previously, alt+shift+n focused the scrollview, which has no name/role, so gave
no meaningful feedback.
BUG=618094

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-06-09 23:44:13]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-06-09 23:45:55]

Description was changed from

==========
Implement update notifications for ChromeVox Next.

- adds a notification on startup alerting the user to a new ChromeVox experience
- adds a skeleton page to be filled with content for the transition
intro/tutorial
- fixes the alert output rule to ignore descendants in the output if the alert
has a name
- adds a new command to open the "new" or "next" announcements/release
notes/tutorial
- fixes two issues with the notification center:
1. the focus wasn't being placed on the first message view item
2. the scroll view parenting the message views was receiving focus but has no
actionable functionality from the keyboard. Removed it from being focusable.
The outcome is when hitting alt+shift+n, an actionable button gets focused with
or wihtout a notification being present (do not disturb gets focus if there are
no messages).
Previously, alt+shift+n focused the scrollview, which has no name/role, so gave
no meaningful feedback.
BUG=618094

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Implement update notifications for ChromeVox Next.

- adds a notification on startup alerting the user to a new ChromeVox experience
- adds a skeleton page to be filled with content for the transition
intro/tutorial
- fixes the alert output rule to ignore descendants in the output if the alert
has a name
- adds a new command to open the "new" or "next" announcements/release
notes/tutorial
- fixes two issues with the notification center:
1. the focus wasn't being placed on the first message view item
2. the scroll view parenting the message views was receiving focus but has no
actionable functionality from the keyboard. Removed it from being focusable.
The outcome is when hitting alt+shift+n, an actionable button gets focused with
or wihtout a notification being present (do not disturb gets focus if there are
no messages).
Previously, alt+shift+n focused the scrollview, which has no name/role, so gave
no meaningful feedback.
BUG=618094

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/9e0f3f0f765bc0625c7988086d8e9ffefd97d337
Cr-Commit-Position: refs/heads/master@{#399051}
==========

[commit-bot: I haz the power, 2016-06-09 23:45:56]

Patchset 5 (id:??) landed as
https://crrev.com/9e0f3f0f765bc0625c7988086d8e9ffefd97d337
Cr-Commit-Position: refs/heads/master@{#399051}