[PPAPI] pp::Var uses the latest version of the PPB_Var API.

[PPAPI] pp::Var uses the latest version of the PPB_Var API.

It previously used version 1.0 even if version 1.1 was available.

BUG=342250
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=250314

[Matt Giuca, 2014-02-10 00:25:31]

Precursor to https://codereview.chromium.org/148213016/ (which adds version 1.2
to Var).

[Matt Giuca, 2014-02-10 00:48:33]

I've refactored it a bit. I think this way makes it easier to add new versions
(there is only a single place in this module where the interface for each method
is selected).

[dmichael (off chromium), 2014-02-10 22:53:08]

lgtm

https://codereview.chromium.org/158563002/diff/60001/ppapi/cpp/var.cc
File ppapi/cpp/var.cc (right):

https://codereview.chromium.org/158563002/diff/60001/ppapi/cpp/var.cc#newcode67
ppapi/cpp/var.cc:67: }
per the style guide (as mentioned in https://codereview.chromium.org/158563002/
), I think this final condition shouldn't be in an else. (Same elsewhere)

[Matt Giuca, 2014-02-10 23:58:09]

https://codereview.chromium.org/158563002/diff/60001/ppapi/cpp/var.cc
File ppapi/cpp/var.cc (right):

https://codereview.chromium.org/158563002/diff/60001/ppapi/cpp/var.cc#newcode67
ppapi/cpp/var.cc:67: }
On 2014/02/10 22:53:08, dmichael wrote:
> per the style guide (as mentioned in
https://codereview.chromium.org/158563002/
> ), I think this final condition shouldn't be in an else. (Same elsewhere)

Done.

[Matt Giuca, 2014-02-11 00:01:08]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2014-02-11 00:07:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mgiuca@chromium.org/158563002/110001

[commit-bot: I haz the power, 2014-02-11 03:45:45]

Change committed as 250314