Tidy up CPDFDOC_Environment.
authorTom Sepez <tsepez@chromium.org>
Thu, 16 Jul 2015 18:09:12 +0000 (11:09 -0700)
committerTom Sepez <tsepez@chromium.org>
Thu, 16 Jul 2015 18:09:12 +0000 (11:09 -0700)
commitfb07e2843dad0774d5842c2b08e7792164efc14a
tree4583acca1bdcf5b60fad79af0b9fa405f192c5d8
parentcc8b4d8763151f51c9fe71ce9b64b2a0d7ddcc88
Tidy up CPDFDOC_Environment.

- untabify as encountered.
- Only put single-statement method in .h file, move more
  complex methods to .cpp (counting an if without braces as
  a single statement, killing braces as needed).
- Move invariant arguments to constructor and make
  corresponding members const.
- Make all members private and add accessor methods.
- Make existing accessor methods const where possible.
- Kill meaningless asserts.
- Add helper functions in place of duplicate code.
- Rename GetCurrentDoc() to GetSDKDocument(), since the class
  has two document members, one of CPDF_Document and one of
  CPDFSDK_Document, making it clear which one you get.
- Simplify some logic with early returns.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1235393002 .
fpdfsdk/include/formfiller/FFL_FormFiller.h
fpdfsdk/include/fsdk_mgr.h
fpdfsdk/src/formfiller/FFL_FormFiller.cpp
fpdfsdk/src/formfiller/FFL_IFormFiller.cpp
fpdfsdk/src/fpdfformfill.cpp
fpdfsdk/src/fsdk_annothandler.cpp
fpdfsdk/src/fsdk_mgr.cpp
fpdfsdk/src/javascript/JS_Object.cpp
fpdfsdk/src/javascript/app.cpp
fpdfsdk/src/javascript/global.cpp