Replace CFX_ByteArray with CFX_ArrayTemplate<uint8_t>

Replace CFX_ByteArray with CFX_ArrayTemplate<uint8_t>

Also replace CFX_Int32Array typedef with CFX_ArrayTemplate<int32_t>.
Removing the typedefs makes subsequent conversion to std::vector<>
easier on a case-by-case basis.

Review-Url: https://codereview.chromium.org/2649563003
Committed: https://pdfium.googlesource.com/pdfium/+/82aa396188ec26f22fe730f4e35b5a54ebffb5dc

[Tom Sepez, 2017-01-20 19:25:15]

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

[commit-bot: I haz the power, 2017-01-20 19:25:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Tom Sepez, 2017-01-20 19:26:43]

tsepez@chromium.org changed reviewers:
+ dsinclair@chromium.org, npm@chromium.org

[Tom Sepez, 2017-01-20 19:26:43]

Dan or npm, for review.

[commit-bot: I haz the power, 2017-01-20 19:45:13]

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

[commit-bot: I haz the power, 2017-01-20 19:45:13]

Dry run: This issue passed the CQ dry run.

[npm, 2017-01-20 19:46:48]

lgtm except I can no longer open the 'View' for BC_HighLevelEncoder.cpp

https://codereview.chromium.org/2649563003/diff/1/xfa/fxbarcode/datamatrix/BC...
File xfa/fxbarcode/datamatrix/BC_HighLevelEncoder.cpp (right):

https://codereview.chromium.org/2649563003/diff/1/xfa/fxbarcode/datamatrix/BC...
xfa/fxbarcode/datamatrix/BC_HighLevelEncoder.cpp:56: 
Why is this file here? Maybe you need to rebase?

[Tom Sepez, 2017-01-20 20:03:45]

On 2017/01/20 19:46:48, npm wrote:
> lgtm except I can no longer open the 'View' for BC_HighLevelEncoder.cpp
> 
>
https://codereview.chromium.org/2649563003/diff/1/xfa/fxbarcode/datamatrix/BC...
> File xfa/fxbarcode/datamatrix/BC_HighLevelEncoder.cpp (right):
> 
>
https://codereview.chromium.org/2649563003/diff/1/xfa/fxbarcode/datamatrix/BC...
> xfa/fxbarcode/datamatrix/BC_HighLevelEncoder.cpp:56: 
> Why is this file here? Maybe you need to rebase?

Something about that file always breaks reitveld.  try the unified diff view
rather than side by side.

[Tom Sepez, 2017-01-20 20:36:45]

The CQ bit was checked by tsepez@chromium.org

[Tom Sepez, 2017-01-20 20:36:45]

The patchset sent to the CQ was uploaded after l-g-t-m from npm@chromium.org
Link to the patchset: https://codereview.chromium.org/2649563003/#ps20001
(title: "re-upload")

[commit-bot: I haz the power, 2017-01-20 20:36:49]

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, 2017-01-20 20:59:50]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484944605166200,
"parent_rev": "6db6fbcdee9f1887ac02c647210bd5013358f12d", "commit_rev":
"82aa396188ec26f22fe730f4e35b5a54ebffb5dc"}

[commit-bot: I haz the power, 2017-01-20 20:59:52]

Description was changed from

==========
Replace CFX_ByteArray with CFX_ArrayTemplate<uint8_t>

Also replace CFX_Int32Array typedef with CFX_ArrayTemplate<int32_t>.
Removing the typedefs makes subsequent conversion to std::vector<>
easier on a case-by-case basis.
==========

to

==========
Replace CFX_ByteArray with CFX_ArrayTemplate<uint8_t>

Also replace CFX_Int32Array typedef with CFX_ArrayTemplate<int32_t>.
Removing the typedefs makes subsequent conversion to std::vector<>
easier on a case-by-case basis.

Review-Url: https://codereview.chromium.org/2649563003
Committed:
https://pdfium.googlesource.com/pdfium/+/82aa396188ec26f22fe730f4e35b5a54ebff...
==========

[commit-bot: I haz the power, 2017-01-20 20:59:53]

Committed patchset #2 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/82aa396188ec26f22fe730f4e35b5a54ebff...