remove some CFX_ArrayTemplate in xfa_ffdocview.h

remove some CFX_ArrayTemplate in xfa_ffdocview.h

Review-Url: https://codereview.chromium.org/2593163002
Committed: https://pdfium.googlesource.com/pdfium/+/d654d7854951e0ed3589fcc34617c7ca593d9e86

[Tom Sepez, 2016-12-21 19:56:00]

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

[commit-bot: I haz the power, 2016-12-21 19:56:08]

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-12-21 20:13:29]

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

[commit-bot: I haz the power, 2016-12-21 20:13:30]

Dry run: This issue passed the CQ dry run.

[Tom Sepez, 2016-12-21 21:12:19]

tsepez@chromium.org changed reviewers:
+ thestig@chromium.org

[dsinclair, 2017-01-02 17:47:13]

dsinclair@chromium.org changed reviewers:
+ dsinclair@chromium.org

[dsinclair, 2017-01-02 17:47:13]

https://codereview.chromium.org/2593163002/diff/1/xfa/fxfa/app/xfa_ffdocview.cpp
File xfa/fxfa/app/xfa_ffdocview.cpp (right):

https://codereview.chromium.org/2593163002/diff/1/xfa/fxfa/app/xfa_ffdocview....
xfa/fxfa/app/xfa_ffdocview.cpp:648: CXFA_WidgetAcc* pCurrentAcc =
What about just doing:

AddCalculateWidgetAcc(pResultAcc)

here as it looks like this is duplicate code to line ~634 above?

[Tom Sepez, 2017-01-03 22:03:49]

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

[commit-bot: I haz the power, 2017-01-03 22:04:00]

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-03 22:04:03]

https://codereview.chromium.org/2593163002/diff/1/xfa/fxfa/app/xfa_ffdocview.cpp
File xfa/fxfa/app/xfa_ffdocview.cpp (right):

https://codereview.chromium.org/2593163002/diff/1/xfa/fxfa/app/xfa_ffdocview....
xfa/fxfa/app/xfa_ffdocview.cpp:648: CXFA_WidgetAcc* pCurrentAcc =
On 2017/01/02 17:47:13, dsinclair wrote:
> What about just doing:
> 
> AddCalculateWidgetAcc(pResultAcc)
> 
> here as it looks like this is duplicate code to line ~634 above?

Done.

[dsinclair, 2017-01-03 22:06:44]

lgtm

[Tom Sepez, 2017-01-03 22:08:07]

The CQ bit was unchecked by tsepez@chromium.org

[Tom Sepez, 2017-01-03 22:08:10]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2017-01-03 22:08:20]

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-03 22:18:08]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483481290515090,
"parent_rev": "72f50f2021cd9f66016c962df8d27fc0c34748cd", "commit_rev":
"d654d7854951e0ed3589fcc34617c7ca593d9e86"}

[commit-bot: I haz the power, 2017-01-03 22:18:10]

Description was changed from

==========
remove some CFX_ArrayTemplate in xfa_ffdocview.h
==========

to

==========
remove some CFX_ArrayTemplate in xfa_ffdocview.h

Review-Url: https://codereview.chromium.org/2593163002
Committed:
https://pdfium.googlesource.com/pdfium/+/d654d7854951e0ed3589fcc34617c7ca593d...
==========

[commit-bot: I haz the power, 2017-01-03 22:18:11]

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