Suppress embedded object output and flush load complete output

Suppress embedded object output and flush load complete output

- Reading the embeddedObject role (not localized) is uneeded noise.

- load completes should always interrupt other output (including utterances left over from the previous page).

BUG=620744, 540800
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/c40225d7f02bbc60b8b761631368dd1de0c25b40
Cr-Commit-Position: refs/heads/master@{#425377}

[David Tseng, 2016-10-12 22:31:51]

Description was changed from

==========
Suppress embedded object output and flush load complete output

- Reading the embeddedObject role (not localized) is uneeded noise.

- load completes should always interrupt other output (including utterances left
over from the previous page).

BUG=
==========

to

==========
Suppress embedded object output and flush load complete output

- Reading the embeddedObject role (not localized) is uneeded noise.

- load completes should always interrupt other output (including utterances left
over from the previous page).

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

[David Tseng, 2016-10-12 22:32:40]

Description was changed from

==========
Suppress embedded object output and flush load complete output

- Reading the embeddedObject role (not localized) is uneeded noise.

- load completes should always interrupt other output (including utterances left
over from the previous page).

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

to

==========
Suppress embedded object output and flush load complete output

- Reading the embeddedObject role (not localized) is uneeded noise.

- load completes should always interrupt other output (including utterances left
over from the previous page).

BUG=620744,540800
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-10-12 22:32:40]

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

[dmazzoni, 2016-10-14 07:26:42]

lgtm

https://codereview.chromium.org/2412613007/diff/1/chrome/browser/resources/ch...
File chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
(right):

https://codereview.chromium.org/2412613007/diff/1/chrome/browser/resources/ch...
chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js:452:
embeddedObject: {
nit: indentation

[David Tseng, 2016-10-14 15:14:13]

The CQ bit was checked by dtseng@chromium.org

[David Tseng, 2016-10-14 15:14:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2412613007/#ps20001
(title: "Address feedback")

[commit-bot: I haz the power, 2016-10-14 15:14:31]

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-10-14 17:35:44]

Description was changed from

==========
Suppress embedded object output and flush load complete output

- Reading the embeddedObject role (not localized) is uneeded noise.

- load completes should always interrupt other output (including utterances left
over from the previous page).

BUG=620744,540800
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Suppress embedded object output and flush load complete output

- Reading the embeddedObject role (not localized) is uneeded noise.

- load completes should always interrupt other output (including utterances left
over from the previous page).

BUG=620744,540800
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-10-14 17:35:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-14 17:40:07]

Description was changed from

==========
Suppress embedded object output and flush load complete output

- Reading the embeddedObject role (not localized) is uneeded noise.

- load completes should always interrupt other output (including utterances left
over from the previous page).

BUG=620744,540800
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Suppress embedded object output and flush load complete output

- Reading the embeddedObject role (not localized) is uneeded noise.

- load completes should always interrupt other output (including utterances left
over from the previous page).

BUG=620744,540800
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[commit-bot: I haz the power, 2016-10-14 17:40:08]

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