Revert "Merge to master: contention over isolate data slots"

Reason for revert: embeddertests failed.
This reverts commit 70bc04b16646c92f221c5aa56831b01d6ec7c1ca.

TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1358263004 .

Merge to master: contention over isolate data slots

Work on this was first performed on the XFA branch, since
it has additional requirements (FXJSE layer) that needed
to be accomodated by the solution.

(cherry picked from commit ed7b2b50aa1744e0bc5a60bef12c61fa91d863b7)
Original Review URL: https://codereview.chromium.org/1351173002 .

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1354593004 .

Use std::set<> to track active event handlers.

This avoids some custom linked-list code. Also note that
we use a local copy to be sure we removed the same thing
that was added no matter how our callees may muck with the
handler.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1352393003 .

Add nonstd::unique_ptr move assigment operator.

std::unique_ptr supports move assignment as in:
  ptr2 = std::move(ptr1);

R=jyasskin@chromium.org

Review URL: https://codereview.chromium.org/1358163002 .

Add missing 'return' keyword.

Discovered with experimental Clang plugin that flags temporary objects
that are immediately destroyed.

patch from issue 1359063003 at patchset 1 (http://crrev.com/1359063003#ps1)

A=mdempsky@chromium.org
R=mdempsky@chromium.org

Review URL: https://codereview.chromium.org/1359103002 .

[Docs] Add wiki content to Markdown docs

BUG=none
R=tsepez@chromium.org, jam@chromium.org, thestig@chromium.org

Review URL: https://codereview.chromium.org/1356323002 .

Remove CJS_RuntimeFactory

The Factory Design Pattern isn't buying us anything here over just new'ing
the object we want.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1360523004 .

More tidy of CJS_Object, kill implicit cast operator

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1349423006 .

Don't pass null isolates to FXJS_ when we have a real isolate.

Kill some now unused functions as a result.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1350703003 .

Fix a #include in fpdf_page_func.cpp.

Not sure why building with gyp was working despite the missing '../' but
it wasn't working in stricter build systems.

BUG=
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1321293003.

Remove several dead functions from fxjs_v8.{cpp,h}.

Re-arrange things in the header so function overloads are
next to each other, and related things are near each other.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1348693003 .

Make pdfium_embeddertests link in debug component builds.

Target v8 has:

          'direct_dependent_settings': {
            'defines': [
              'V8_SHARED',
              'USING_V8_SHARED',
            ],
          },

For this to work, targets using v8 headers have to depend on v8
directly, else the V8_EXPORT macro won't work. Add a direct
dependency on v8.

BUG=none
TEST=do a debug component build of pdfium_embeddertests on Win,
should work without linker errors

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1349843002 .

Fix build broken at 506df426d5d6.

Move header file includes when usage moved.
Windows-only section missing a ")".

TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1349783003 .

Ensure functions in FXJS_V8 are prefixed by FXJS_.

Currently, its hard to tell which functions come from the JS_
layer at fpdfsdk/include/javascript vs. which functions come
from the FXJS_V8 layer at fpdfsdk/include/jsapi.  Until we
take up the task of using namespaces, at least make the
prefix consistent.

Move objects out of FXJS_V8 that are really part of JS_.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1347833002 .

Check for empty embedder data before using it in PDFium JS bindings.

This was guarded by an assert, but the path is hit.

BUG=528015
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1345923002 .

gyp_pdfium should use ninja by default.

R=thakis@chromium.org, tsepez@chromium.org

Review URL: https://codereview.chromium.org/1330173002 .

Fix typo from commit a9d4bc5. (Autocomplete fail)

TBR=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1345883003 .

Fix build on Windows after commit 0b3c8f7.

MSVS can't figure out the value of strlen(kConstString) at compile time.

TBR=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1342263002 .

Implement FPDFAction_GetFilePath().

The API is the same as the Foxit version, except the encoding is
specified as UTF-8 instead of local encoding.

Also remove CPDF_LWinParam since it's unused.

BUG=chromium:517713
R=jun_fang@foxitsoftware.com

Review URL: https://codereview.chromium.org/1335373002 .

Fix build broken at 09ed30750282

Wrong parameter set to nullptr during one of the CL revisions.

TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1344483005 .

Refactor fxjs_v8 and add embeddertests for it.

This forces the layer defined by fxjs_v8.h to be (more)
self-contained, so that it can be tested apart from the
CJS_* objects (in fpdfsdk/{src,include}/javascript. This
implies the array buffer allocator must be part of fxjs_v8.

One wrinkle is that we'd like to be able to test an isolate
upon which no native objects have been added, so some
initialization that would have occurred as part of object
definition must be made explicit.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1338073002 .

Fix build broken at ac8fda05418b on windows

|constexpr| not supported on windows.

TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1347723002 .

Add move ctor to nonstd::unique_ptr.

R=jyasskin@chromium.org

Review URL: https://codereview.chromium.org/1338383002 .

Cleanup JBig2_GeneralDecoder.cpp.

- FX_Alloc() can't fail and return.
- Use unique_ptr / remove gotos.
- NULL -> nullptr.
- Combine common code.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1337823003 .

Cleanup: Fix a typo. s/Processive/Processing/

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1338573002 .

Add a missing #include that's needed for call to FXSYS_tolower.

BUG=
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1329343002.

Get CJS_RuntimeFactory out of the CJS_GlobalData management business.

First part of getting rid of CJS_RuntimeFactory.  The factory design
pattern isn't appropriate here since we only ever make one kind of
object.

CJS_GlobalData is now perfectly capable of managing itself through
internal ref counts. I'm philosophically opposed to keeping ref-counts
outside the object (do you hear me std::shared_ptr, you're bad!)

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1338993005 .

Convert some unneeded gotos into returns.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1332193002 .

Cleanup casting of FX_Alloc() return values.

Also convert some FX_AllocOrDie() calls to FX_Alloc().

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1332173002 .

Fix strings, remove stringify macros, void return types for Consts.h.

Replace multiple #defines of the same strings with externs.
Fix strings mangled by interaction of # and clang-format.
Remove macros as possible.
Make more JS_ functions void and simplify.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1342433002 .

Rename Init methods to more accurately describe purpose

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1335763002 .

Guard against null image data in CJBig2_GRRDProc.

Credit to karl at skomski.com for the initial version of the CL.

BUG=527174
R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1305033006 .

Remove CJBig2_Object, CJBig2_Module, and friends.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1326953006 .

Remove some abstractions in fxjs_v8.h.

It's too hard to keep mapping between v8 and fx abstractions; the lack
of transparency prevents those skilled in v8 only from working on this
code.

Apparently, the original intention was to confine v8 types to
fpdfsdk/{include,src}/jsapi, but fpdfsdk/{include,src}/javascript
is already well-polluted with v8 types.

Also remove no-op JS_SetThisObj().

Also remove unused ParserParams() [noticed because it was incorrectly
passing handles as pointers].

Also remove cast operator from CJS_Runtime and call GetIsolate()
explicitly.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1332973002 .

Fix JS_GetArrayElement to not return an empty handle on success

BUG=chromium:528376
R=ulan@chromium.org, tsepez@chromium.org

Review URL: https://codereview.chromium.org/1337433002 .

Resolve bad cast to CJS_EventHandler

At line 2026 in pdfsdk/src/javascript/PublicMethods.cpp,
the writer wants to perform type cast like line 2027 but
he made a mistake to make type conversion on m_pValue.
Even at line 2027, it's redundant to make type conversion.
The returned type of pContext->GetEventHandler() is
CJS_EventHandler*.

BUG=529310
R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1325753007 .

Remove existing generated images before running a test.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1314443007 .

Fix heap use after free in CPDFSDK_Annot::GetPDFAnnot.

Use two seperate loops to kill current focus annot and to release annots
in current page. Loop to kill current focus annot is run first, so it
will not access deleted annots.

BUG=507316

R=tsepez@chromium.org

TEST=Reproduction steps mentioned in issue 507316 should not crash
     chrome.
     Unit test added to pdfium.
     Run pdfium_embeddertests.exe.

Review URL: https://codereview.chromium.org/1312313006 .

Test files for issue 507316
Upload pdf test files for codereview 1312313006.

BUG=507316
R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1325533004 .

Beef up app_props.in and tidy app.cpp.

Some of the values returned are dubious; capture the
current state of affairs.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1317393003 .

Change the JS app.viewerType() return value to "pdfium"

BUG=pdfium:181
R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1331633002 .

Remove unneeded checks for CPDF_Object::GetDict() return values.

CPDF_Object::GetDict() always returns a valid pointer for dictionaries.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1317733007 .

Fix typos, nits and remove dead code in fpdf_text_int.cpp.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1316643004 .

Remove CJBig2_Module::JBig2_Error and friends.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1327983003 .

Make a bunch of JBig2 classes independent of CJBig2_Object.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1319713003 .

Turn a failing assert into an actual check.

BUG=522131
R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1327913002 .

Change fixup_pdf_template.py to open files in binary mode.

Fix for the pixel test failure on Windows from commit 870b5b6793fa.
Seems to work for me on Windows.

TBR=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1312493006 .

Remove dead JBig2 code.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1328643002 .

Reapply Foxit's libopenjpeg modifications.

They were lost in commit d53e6fd.

BUG=pdfium:168
R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1196523002 .

Cleanup dead code in CPDF_DIBSource::LoadJpxBitmap() and friends.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1310603006 .

Remove IPDFSDK_AnnotHandler interface.

It's pointless to have an abstract interface and its only concrete
implementation defined in the same header.  We can de-virtualize along
the way and get a small savings.

Rename CPDFSDK_BFAnnotHandler along the way, since its the only kind
of AnnotHandler, and I can't imagine what BF stands for.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1313733013 .

CPDFSDK_MediaActionHandler is unused.

It's never instantiated, and any pointers to it are always NULL.
Also kill an unused arg to CPDFSDK_ActionHandler ctor along the way.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1329583004 .

CJS_Context::compile unused

This, in turn, allows us to get rid of the DoJob method and its clumsy
mode parameter conditional.

Add explicit and consts, and remove pointless asserts along the way.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1318543012 .

Fix two issues shown by bug 489995

- Handle NULL in buffer operator<< under JS mailForm() calls.
- Ensure correct type in JS addIcon() calls.

BUG=489995
R=brucedawson@chromium.org, thestig@chromium.org

Review URL: https://codereview.chromium.org/1327473002 .

Move configs out of targets in GN build files.

I'm trying to disallow this since it's confusing. It looks like it provides
scoping for the inner config, but it is actually no different than declaring
the config at the top level. For clarify, all configs and targets should be
declared at the top level.

R=jam@chromium.org

Review URL: https://codereview.chromium.org/1328443003 .

Fix infinite loop for objects that reference themselves.

BUG=pdfium:193
R=jun_fang@foxitsoftware.com

Review URL: https://codereview.chromium.org/1306793002 .

Replace signed char with int8_t. Fix Linux ARM build after r7830e9e.

TBR=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1321883002 .

Check array bounds for opj_dwt_decode_1() and friends.

Based on an earlier patch by jun_fang@foxitsoftware.com.

BUG=450844
R=jun_fang@foxitsoftware.com

Review URL: https://codereview.chromium.org/1320443003 .

Fix some -Wmaybe-uninitialized errors.

- Break up CPDF_DIBSource::DownSampleScanline() into smaller functions.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1287843004 .

Remove a bunch of dead JBIG2 code.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1293393002 .

Extern in .cpp file is a code smell, part 2.

Fixes the remaining issues, except for JBIG2/Skia ...

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1298393003 .

Extern in .cpp files is a code smell.

Part 1.  Move to headers so compiler can type check against
the definitions.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1299963002 .

Fix mac build after commit e1ce94e.

TBR=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1297303002 .

Cleanup CFX_Font a bit.

- Make methods const.
- Make variables non-public.
- Remove LoadFile() method.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1293973002 .

Remove unused code warnings found by clang.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1297593005 .

FX_CMapDwordToDword considered harmful.

Lookups are log(n), but random insertions could result in n^2
behaviour.  Replace with maps and sets.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1289703003 .

Remove more dead code from fx_basic.h

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1294683003 .

Clean up IFX_BidiChar

- Replace IFX_BidiChar with just CFX_BidiChar
- Document implementation
- Change out parameters to pointers
- Remove dead code
- Add an enum for bidi directions
- Move several externs to a header
- Add unit tests

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1197643002 .

Fix fix fix fix of breakage on mac.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1296383004 .

Fix fix fix of mac build breakage at 9cf44c2e

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1296403002 .

Fix fix of mac build breakage at 2a2a6aa

TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1293153003 .

Fix mac build breakage at ce4ffb8.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1287053006 .

CFX_MapByteStringToPtr considered harmful.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1297723002 .

Cleanup CFX_UnicodeEncoding and remove IFX_FontEncoding.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1296753003 .

Fix more sign comparison errors.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1290383003 .

js_global_data and global_alternate violate naming rules.

They look too much like locals; use JSGlobalData and JSGlobalAlternate
instead.  Kill some commented out code.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1289903004 .

Suppress -Wswitch for pdfium's fxge target for now.

BUG=pdfium:188
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1301473003 .

Fix -Wunused-function warnings on Windows.

BUG=505316
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1293033002 .

Fix a incorrectly placed comment from 0f6b51c

TBR=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1288053005 .

Clean: Fix some unneeded semi-colons and bad spacing.

TBR=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1284193005 .

Add new public APIs to find the z-order for links and widgets.

- Implement FPDFLink_GetLinkZOrderAtPoint().
- Implement FPDFPage_FormFieldZOrderAtPoint().
- Mark FPDPage_HasFormFieldAtPoint() as deprecated.
- Modify CPDF_LinkList and CPDF_InterForm to support new APIs.
- Clean up dead code in CPDF_LinkList and CPDF_InterForm.

BUG=chromium:515837
R=jun_fang@foxitsoftware.com, tsepez@chromium.org

Review URL: https://codereview.chromium.org/1278053004 .

Don't bother checking pointers before delete[] and FX_Free().

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1297713003 .

Use override in more classes in fpdfsdk/

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1287193005 .

Use override in more classes in core/

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1292613003 .

Cleanup: Remove unused CPDF_Metadata::m_pDoc.

Also remove unused kAddinNameCJK variable.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1296513003 .

CPDF_ModuleMgr::m_SecurityHandlerMap only used by dead code.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1289693008 .

Fix build after commit 22ece1a. Too many if's.

TBR=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1294723002 .

Remove last uses of FOXIT_CHROME_BUILD.

BUG=pdfium:46
R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1294693002 .

Cleanup: s/Torelance/Tolerance/

R=tsepez@chromium.org
TBR=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1294713002 .

OutputText() is dead code.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1291213003 .

Tidy up JS_Object.h and JS_Object.cpp.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1293673003 .

Kill JS_TIMER_MAPARRAY

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1286383004 .

Cleanup: Remove unused CPDF_Stream::Clone().

Cloning a CPDF_Stream actually happens via CPDF_Object::Clone().

Transitively, remove:
- GetStreamFilter()
-- all the filters.

R=jam@chromium.org

Review URL: https://codereview.chromium.org/1288543002 .

Make standalone PDFium DEPS pull clang-format.

R=thakis@chromium.org

Review URL: https://codereview.chromium.org/1297453002 .

Remove if checks after new.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1287863002 .

Suppress -Wunused-function for pdfium's third-party libraries.

Also add a missing header file.

BUG=505316
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1285123003 .

Allow external font-path configuration from pdfium_test.

R=thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/d8b5e73d8609b74e6a995ee1768d20d47bd4b089

Review URL: https://codereview.chromium.org/1268323004 .

Revert "Allow external font-path configuration from pdfium_test."

This reverts commit d8b5e73d8609b74e6a995ee1768d20d47bd4b089.
Broke corpus tests

TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1292153002 .

Allow external font-path configuration from pdfium_test.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1268323004 .

Fix a small leak in CPDF_DataAvail::CheckTrailer().

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1278713003 .