Return the correct channelCountMode and channelInterpretation

Return the correct channelCountMode and channelInterpretation

Instead of returning the current mode or interpretation, return the
value used in the setter.  Eventually, the real mode will get set to
the desired value.

The members m_channelCountMode and m_channelInterpretation are also
made private to force use of the setters and getters.  A setter is
added too so that the internal state is maintained.

BUG=636576
TEST=channel-mode-interp-basic.html

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

[Raymond Toy, 2016-08-25 15:55:42]

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

[commit-bot: I haz the power, 2016-08-25 15:56:05]

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-25 17:21:44]

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

[commit-bot: I haz the power, 2016-08-25 17:21:45]

Dry run: This issue passed the CQ dry run.

[Raymond Toy, 2016-08-25 18:13:44]

Description was changed from

==========
Return the correct channelCountMode and channelInterpretation

Instead of returning the current mode or interpretation, return the
value used in the setter.  Eventually, the real mode will get set to
the desired value.

BUG=636576
TEST=
==========

to

==========
Return the correct channelCountMode and channelInterpretation

Instead of returning the current mode or interpretation, return the
value used in the setter.  Eventually, the real mode will get set to
the desired value.

The members m_channelCountMode and m_channelInterpretation are also made private
to force use of the setters and getters.  A setter is added too so that the
internal state is maintained.

BUG=636576
TEST=channel-mode-interp-basic.html
==========

[Raymond Toy, 2016-08-25 20:09:55]

rtoy@chromium.org changed reviewers:
+ hochchan@chromium.org

[Raymond Toy, 2016-08-25 20:09:56]

PTAL.

[Raymond Toy, 2016-09-01 16:36:41]

Description was changed from

==========
Return the correct channelCountMode and channelInterpretation

Instead of returning the current mode or interpretation, return the
value used in the setter.  Eventually, the real mode will get set to
the desired value.

The members m_channelCountMode and m_channelInterpretation are also made private
to force use of the setters and getters.  A setter is added too so that the
internal state is maintained.

BUG=636576
TEST=channel-mode-interp-basic.html
==========

to

==========
Return the correct channelCountMode and channelInterpretation

Instead of returning the current mode or interpretation, return the
value used in the setter.  Eventually, the real mode will get set to
the desired value.

The members m_channelCountMode and m_channelInterpretation are also 
made private to force use of the setters and getters.  A setter is 
added too so that the internal state is maintained.

BUG=636576
TEST=channel-mode-interp-basic.html
==========

[Raymond Toy, 2016-09-01 16:36:56]

Ping

[hongchan, 2016-09-01 18:56:51]

On 2016/09/01 16:36:56, Raymond Toy wrote:
> Ping

Weirdly enough this CL does not show up in my 'incoming review' section. That's
why I missed it.

[hongchan, 2016-09-01 18:57:16]

On 2016/09/01 18:56:51, hoch wrote:
> On 2016/09/01 16:36:56, Raymond Toy wrote:
> > Ping
> 
> Weirdly enough this CL does not show up in my 'incoming review' section.
That's
> why I missed it.

Oh, your reviewer is 'hochchan' which is not me technically. Heh.

[Raymond Toy, 2016-09-01 19:52:16]

rtoy@chromium.org changed reviewers:
+ hongchan@chromium.org
- hochchan@chromium.org

[Raymond Toy, 2016-09-01 19:53:02]

On 2016/09/01 at 18:57:16, hongchan wrote:
> On 2016/09/01 18:56:51, hoch wrote:
> > On 2016/09/01 16:36:56, Raymond Toy wrote:
> > > Ping
> > 
> > Weirdly enough this CL does not show up in my 'incoming review' section.
That's
> > why I missed it.
> 
> Oh, your reviewer is 'hochchan' which is not me technically. Heh.

Ugh.  The UI for the new review tool doesn't complete names the same as
rietveld.

[hongchan, 2016-09-01 20:38:55]

lgtm

[Raymond Toy, 2016-09-01 20:40:14]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2016-09-01 20:40:47]

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-09-01 22:06:12]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-01 22:09:28]

Description was changed from

==========
Return the correct channelCountMode and channelInterpretation

Instead of returning the current mode or interpretation, return the
value used in the setter.  Eventually, the real mode will get set to
the desired value.

The members m_channelCountMode and m_channelInterpretation are also 
made private to force use of the setters and getters.  A setter is 
added too so that the internal state is maintained.

BUG=636576
TEST=channel-mode-interp-basic.html
==========

to

==========
Return the correct channelCountMode and channelInterpretation

Instead of returning the current mode or interpretation, return the
value used in the setter.  Eventually, the real mode will get set to
the desired value.

The members m_channelCountMode and m_channelInterpretation are also
made private to force use of the setters and getters.  A setter is
added too so that the internal state is maintained.

BUG=636576
TEST=channel-mode-interp-basic.html

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

[commit-bot: I haz the power, 2016-09-01 22:09:28]

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