Pepper: Make C++ wrappers for PPB_AudioConfig backwards compatible.

Pepper: Make C++ wrappers for PPB_AudioConfig backwards compatible.

This makes the wrappers work transparently with both versions 1.1 and 1.0 of the
interface.

(The compatibility is needed for Pepper Flash to use the wrappers; Pepper Flash
needs to work with M18 until M19 ships, at which point we can remove the
1.0-compatibility.)


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

[viettrungluu, 2012-03-06 20:51:25]

yzshen -> review, brettw -> OWNERS stamp

[yzshen1, 2012-03-06 20:54:34]

lgtm

https://chromiumcodereview.appspot.com/9617018/diff/1/ppapi/cpp/audio_config.cc
File ppapi/cpp/audio_config.cc (right):

https://chromiumcodereview.appspot.com/9617018/diff/1/ppapi/cpp/audio_config....
ppapi/cpp/audio_config.cc:40: if (has_interface<PPB_AudioConfig_1_0>()) {
You need an 'else' here. haha.

[viettrungluu, 2012-03-06 22:17:21]

https://chromiumcodereview.appspot.com/9617018/diff/1/ppapi/cpp/audio_config.cc
File ppapi/cpp/audio_config.cc (right):

https://chromiumcodereview.appspot.com/9617018/diff/1/ppapi/cpp/audio_config....
ppapi/cpp/audio_config.cc:40: if (has_interface<PPB_AudioConfig_1_0>()) {
On 2012/03/06 20:54:34, yzshen1 wrote:
> You need an 'else' here. haha.

Done, thanks. I'm apparently incapable of making trivial changes correctly
today! (Or maybe not just today....)

[brettw, 2012-03-07 00:17:46]

lgtm

[commit-bot: I haz the power, 2012-03-07 01:13:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/viettrungluu@chromium.org/9617018/5003

[commit-bot: I haz the power, 2012-03-07 03:13:32]

Change committed as 125310