Don't pay for filename/line number lookup unless you need to.

This wasn't a regression; the code's always been like this. But this shows up
in profiles of anything doing a lot of synchronized stuff, even when there's
no contention.

There are two awkward cases. One is that the old code used to offer a variety
of special-case messages for failures to find the current frame, save area,
and Method*. I assume this was just to help in debugging and shouldn't happen
in practice, so I don't think we've lost anything there. The other case was
that on unlocking, we used to explicitly say "unlocked". But I'm not sure we
could get into a situation where we'd be reporting contention on a monitor
that wasn't locked when we tried to lock it. So I think that's okay too.

Change-Id: Ib4401c771f717e9c8cc9c4e5346ec7a5f46a1636

Merge "DO NOT MERGE: Optimize IndirectRefTable." into ics-mr1

Don't malloc between fork and exec

This was causing hangs in DEX file loading.

Bug 5329833

Change-Id: I19e9e0624f3823cc771c71a4b4ac177d57343c58

am 3931f798: Fix cleanup (cherry-picked from ics-mr1)

* commit '3931f79801b67577bf8ac12d0b3fb5b4aeaaf766':
  Fix cleanup (cherry-picked from ics-mr1)

Fix cleanup
(cherry-picked from ics-mr1)

The code was attempting to null out a struct member after freeing
the struct.

Also, changed the order of directory permission tests so that
writable comes first.  Somehow "dalvik-cache directory not
writable" seems more direct than "not readable", since the code
isn't generally interested in reading the directory.

Bug 5549907

Change-Id: If737ab822b356aae98e47292d21946e33a04342b

am e3d9cde4: Disable JIT single-stepping resume mode.

* commit 'e3d9cde40f96bf40dd4ad3d495949869bde5dd1e':
  Disable JIT single-stepping resume mode.

Disable JIT single-stepping resume mode.

BUG: 5551114
Change-Id: Id8ee75be5e71ee258994235952871ad181024e56

Fix cleanup

The code was attempting to null out a struct member after freeing
the struct.

Also, changed the order of directory permission tests so that
writable comes first.  Somehow "dalvik-cache directory not
writable" seems more direct than "not readable", since the code
isn't generally interested in reading the directory.

Bug 5549907

Change-Id: Ie412ff6c0870357b92e53606f70cb9407e04e7cf

DO NOT MERGE: Optimize IndirectRefTable.

Removed unused debugging code that maintained pointers to the
4 previous objects in each slot.

Promoted the serial number property of a slot to be first class.
This is a sufficiently cheap and useful check to perform that we
might as well always do it.

Changed get() and remove() so they check the validity of the
references at the same time as they retrieve them.  They're
also a bit smarter about checking the reference kind given
that they can assume that the table will only contain references
of one kind.  The checks are now much more consistent.
For example, remove() used to check whether an index was
stale only in the case where it was not removing the top entry
(now it always checks).

Made add() return NULL instead of dvmAbort()'ing in the case
where the table overflowed and ensure we don't blow away the
table in that cases.  This change resolves an issue with
TestIndirectRefTable.cpp which deliberately overflows the table
and expects add() to return NULL (as documented!).  As it
happens, the add() method is called in exactly 3 places in
Jni.cpp.  In each of those cases, the code was written to
handle a NULL result and only in the case of adding a weak
global reference did it not abort.  Fixed the weak global
reference case to be consistent with the others.

Changed the signature of contains() to take an Object* since
that's what we're actually looking for inside the table.

Added a couple of calls to dump() in TestIndirectRefTable.cpp
for visual inspection of its correctness.

Performance as measured by TestIndirectRefTable on same hardware.

Old implementation:
- Add/remove 100 objects FIFO order, 100000 iterations, 0.023ms / iteration
- Add/remove 100 objects LIFO order, 100000 iterations, 0.020ms / iteration
- Get 100 objects, 100000 iterations, 0.009ms / iteration

New implementation:
- Add/remove 100 objects FIFO order, 100000 iterations, 0.010ms / iteration
- Add/remove 100 objects LIFO order, 100000 iterations, 0.009ms / iteration
- Get 100 objects, 100000 iterations, 0.002ms / iteration

Cherry-pick from master.

Conflicts:

vm/IndirectRefTable.cpp

Change-Id: I157f3c1ba598137222878b8e6a5890efb744fe76

am b1e2271b: Fix single-stepping resume mode in the interpreter.

* commit 'b1e2271b51cfb1f264b8966f1151652767259751':
  Fix single-stepping resume mode in the interpreter.

Fix single-stepping resume mode in the interpreter.

(cherry-picked from master)

When a resume attempt is cancelled due to other pending request make sure
the old native resume PC is cleared. Otherwise the JIT code cache may be
re-entered with mismatching Dalvik PC.

Also fix a code bloat problem where single-step count is not set properly
after executing a return instruction.

BUG: 5208786

Change-Id: I39db1633dcda1f464311c878ded4b5695a9b7ffd

Fix single-stepping resume mode in the interpreter.

(cherry-picked from master)

When a resume attempt is cancelled due to other pending request make sure
the old native resume PC is cleared. Otherwise the JIT code cache may be
re-entered with mismatching Dalvik PC.

Also fix a code bloat problem where single-step count is not set properly
after executing a return instruction.

BUG: 5208786

Change-Id: I54775215b11eae29ccdb6111dc0fdfa99e41e08d

Merge "Provide debugging info should extended opcode lookup fail."

Provide debugging info should extended opcode lookup fail.

Bug: http://code.google.com/p/android/issues/detail?id=20814
Change-Id: Iedaf07721c29dd3d24827c3f7a953b7371c9735d

Avoid deadlock.

dvmIsHeapAddress is the pointer validity checker for use outside the GC. If
it doesn't take the heap lock, it risks looking at data structures that are
changing under its feet. If it does take the heap lock, it risks deadlock
between a thread doing an explicit GC and the signal catcher trying to dump
threads (causing the GC thread to be suspended while holding the heap lock).

Calling back into managed code while holding the heap lock and with other
threads resumed sounds like an inherently bad idea to me, but that's a battle
for another day.

With this change, we can handle SIGQUIT while we're in ReferenceQueue.add
as a result of an explicit GC in a system doing concurrent collections, so
the only known problem is fixed.

Bug: 5425802
Change-Id: I42d434d5ea3ffbcb77a4c544b81b08a4c7364a16

Merge "Leave up to 512 chars of gDvm.lastMessage on dvmAbort's stack frame."

Leave up to 512 chars of gDvm.lastMessage on dvmAbort's stack frame.

Bug: 5372634
Change-Id: I019a059e5a989de3dfc9e2f5e78f7caa7168cf75

Merge "Reordered some checks and added more to the verifier."

Merge "Revert "Use the card table to scan the immune region of the heap.""

Revert "Use the card table to scan the immune region of the heap."

This reverts commit 6c355e53332502314c3d82a7afcf898d66118f27.

Conflicts are resolved in:

vm/alloc/CardTable.cpp
vm/alloc/HeapBitmap.cpp
vm/alloc/HeapInternal.h
vm/alloc/HeapSource.cpp
vm/alloc/MarkSweep.cpp

Tested with overnight monkey runs + Delaunay.

Bug: 5037417
Change-Id: I400eff8542d0bddd865ce570a479f7750c1ba484

Merge "Don't say UTF-8 in Dalvik's CheckJNI when we mean Modified UTF-8."

Don't say UTF-8 in Dalvik's CheckJNI when we mean Modified UTF-8.

Change-Id: I4130e7f92275275a6d6f782e1ad979fbf9197a8e

Reordered some checks and added more to the verifier.

The verifier checks for 2 errors in put instructions. First, if the
field can't be resolved, the instruction is overwritten to throw a
verifier error. Second, if the data types of the field and instruction
mismatch, the verifier rejects the whole class. This change swaps the
order of the checks, so that if both errors occur, the class is
rejected.

Also, checks were added to prevent float values from being used as array
indexes or arguments to if statements.

Change-Id: I7b122f4ebc4043266be4466e10679c6319a788ef

Fixed dx verifier issue when an error is detected.

The code needed to break out properly when an error was detected to
prevent it from causing even more problems.

Change-Id: If497655a406689aeb1b4aa6523421cc22f2b7593

Fix a typo in unused code.

Change-Id: Ied11a79337a3cc795428dfde29e405ec0bbcb98c

Fix memory barriers (Issue 3338450)

Add extra memory barrier on volatile stores.

Change-Id: Id4a4750cdfc910eda2f0b44ead0af2a569b5735e

Don't spam the logs with thread priority information.

Ten minutes of logs contained 108 lines of 'Elevating priority from 10 to 0'
and 108 lines of 'Reset priority to 10'. This provides little value.

Change-Id: Id96e8304238aa3344af4dd6342ace575534acc0f

Attempt to fix dalvik heap corruption.

This is bccheng's find, as reported here:
http://b/5208786#ISSUE_HistoryHeader49

Bug: http://b/5208786
Change-Id: I1616bac1ef6c197f723656187a784fc6838975d4

Fix document formatting error (html tags)

Change-Id: I57d10b69f400ce3ffda1a26dc632a4122a4f3d70

Merge "Dx reg allocator fix for accidental long/double clobbering."

Merge "Rename register to conform to instruction format"

Rename register to conform to instruction format

For instruction format 35c, 41c, 52c, and 5rc, the register
name was different from the Dalvik Instruction Format document,
and InstrUtils.cpp of libdex.  This commit updates the name,
so that they will look consistent.

Change-Id: I857b1f9c1ac7178dc147d0019df589c9c6d102e1

Fix a corner-case dex merging bug.

When doing a null merge (merging a dex file with an empty file) word-alignment
means that the output may be two bytes longer than the input. Always include the
rounded-up length in the bytes allocated to a section.

com.android.dx.util.DexException: Section limit 669894 exceeded by type list
at com.android.dx.io.DexBuffer$Section.ensureCapacity(DexBuffer.java:541)
at com.android.dx.io.DexBuffer$Section.writeShort(DexBuffer.java:574)
at com.android.dx.io.DexBuffer$Section.writeTypeList(DexBuffer.java:636)
at com.android.dx.merge.DexMerger$3.write(DexMerger.java:439)
at com.android.dx.merge.DexMerger$3.write(DexMerger.java:424)
at com.android.dx.merge.DexMerger$IdMerger.mergeUnsorted(DexMerger.java:321)

Change-Id: I67b81ba8c837b0307e313f4012a051b022b25db2

am 4130f726: am 1dd1d924: am e4e76feb: Merge "Set DX stack size to 1M by default on Windows."

* commit '4130f726d82493748bf7252da631e1f315f2538c':
  Set DX stack size to 1M by default on Windows.

am 1dd1d924: am e4e76feb: Merge "Set DX stack size to 1M by default on Windows."

* commit '1dd1d9246c0804d1b4fda955c56a6d89c989ec6a':
  Set DX stack size to 1M by default on Windows.

am e4e76feb: Merge "Set DX stack size to 1M by default on Windows."

* commit 'e4e76feb6aad0d0161fabe12cb7d97e665f7b109':
  Set DX stack size to 1M by default on Windows.

Merge "Set DX stack size to 1M by default on Windows."

Dx reg allocator fix for accidental long/double clobbering.

Addresses this bug: http://b/issue?id=5215643.

Change-Id: I82613c38b0a6247d071aa6c5cc90bf8b60d5d84b

Set DX stack size to 1M by default on Windows.

Bug: http://code.google.com/p/android/issues/detail?id=15181

Summary of bug: some user reported issues running dx.bat and
Fadden figured out the VM's stack size default was too small.
Using -Xss1m fixed it.

This CL introduces a default of -Xss1m and the usual mechanism
to override it using -JXssNm (like for -JXmx...)

Change-Id: I9769316c1708f08c91b93e07bd6d713a10561ea2

Removed parsing and output for unused --target-api switch in dx.

This is cherry-picked from dalvik-dev.

Change-Id: I74627e4c261e086e8c2c7cf38591cd3e9f5707e6

Merge "Fix JNIEXPORT now people are using -fvisibility."

Permit code size to grow when merging dex files.

The throws clauses use variable-length encodings (sleb and uleb)
which will have different lengths when the number of types grows.

Bug: http://b/5127963
Change-Id: I50598a807596955e458d4ec87a730e62e12168d9

Fix JNIEXPORT now people are using -fvisibility.

This is a backport from dalvik-dev.

Change-Id: If6001b580bd9b36f3845aaa115333d69682f3a92

Fix two data races in the monitor code.

First, the release of a lock was not preceded by a memory barrier.
This would allow stores within the critical section to move outside
the critical section violating the monitor invariant.

Second, the thin lock value was often not loaded through volatile
pointer.  This provides the compiler with the freedom to reload the
thin lock with a different value causing erroneous updates to the lock
word.

Bug: 5107390

Merge "Dexopt verifier checks that int/float and double/long uses do not mix."

Enhance CheckJNI to recognize when an type-incompatible jfieldID is used to get/set a field.

This would have found a bug that broke setSoTimeout in libcore, and did find
a hitherto-unknown bug in android.text.format.Time.

Bug: 5162851
Change-Id: I62f2a0db3adb1cd0e4616075f6d4efaee393de30

Don't wrap the actual type in parens if its null.

Otherwise the exception message has unnecessary parens:
"expected receiver of type java.lang.String, but got (null)"

Change-Id: Iacd806d018019784afa6e9f25f7c039d9ca18fae

Dexopt verifier checks that int/float and double/long uses do not mix.

Added new types to dexopt indicating a type is derived from a constant,
which may or may not be floating point. After such a value is used, its
type can be determined, and the verifier enforces that it is always used
that way afterwards.

Change-Id: I150deaa162aec17bcc9c3e3200bc0350ae5b3f24

am be3942f2: (-s ours) am 3245c2e4: am 5017f3f2: Merge "Support wrapping app processes to inject debug instrumentation. Bug: 4437846"

* commit 'be3942f2179d5f2270b757a792ed021eb2521b46':
  Support wrapping app processes to inject debug instrumentation. Bug: 4437846

am 8ddfe8b4: (-s ours) am a0533621: am 9df264db: Merge "Add a dual clock profiler tracing mode."

* commit '8ddfe8b44e0794a0c182046144af8c3c842686c2':
  Add a dual clock profiler tracing mode.

am 3245c2e4: am 5017f3f2: Merge "Support wrapping app processes to inject debug instrumentation. Bug: 4437846"

* commit '3245c2e4008b40e05db44ef7c897d0aa7b88c1d7':
  Support wrapping app processes to inject debug instrumentation. Bug: 4437846

am a0533621: am 9df264db: Merge "Add a dual clock profiler tracing mode."

* commit 'a05336218332abf2753c63a50ede7551afdab6b5':
  Add a dual clock profiler tracing mode.

am 5017f3f2: Merge "Support wrapping app processes to inject debug instrumentation. Bug: 4437846"

* commit '5017f3f2b7c178761b919f22c02617fe6fe9468d':
  Support wrapping app processes to inject debug instrumentation. Bug: 4437846

am 9df264db: Merge "Add a dual clock profiler tracing mode."

* commit '9df264db156d091f7c1de30295f4c5d330ca3694':
  Add a dual clock profiler tracing mode.

Merge "Support wrapping app processes to inject debug instrumentation. Bug: 4437846"

Merge "Add a dual clock profiler tracing mode."

Take the heap lock in dvmIsHeapAddress.

Bug: 5049447
Change-Id: I9df6e4116efa44088d37ac0ff4455a7876535e1e

Reconcile with honeycomb-mr2-release

Change-Id: I67bdc33aeb248a10c08e1a9bbc1938101304f29d

Rollback of dx change to prevent methods from being overwritten.

Rolls back: I999013858501e7de3df27f82199d377fb1d034ea

Change-Id: Id67611b0153d49a81a2fbe0aaa8a7c9a9784987c

Dx fit to prevent method arguments from being overwritten by temps.

This change keeps code debuggable by preventing method arguments from
being reused to hold temporary values. This was a problem especially for
static methods.

Change-Id: I999013858501e7de3df27f82199d377fb1d034ea

Support wrapping app processes to inject debug instrumentation.
Bug: 4437846

Remove "adbd disconnected" log spam

Change-Id: I532433a5ee3f23bad8dc5fdb0f398253f2b6eb8b
Signed-off-by: Mike Lockwood <lockwood@android.com>

Only allocate slot_data_ as needed.

Bug: 5052119
Change-Id: I41434b111c1f91cf1beb3125af6e81cab258cb92

Merge "Don't use dvmIsValidObject outside the GC."

Fix memory leak in mapCentralDirectory.
Bug: 4984352

mapCentralDirectory was allocating a buffer of size 65557
that it passed to mapCentralDirectory0.  Then mapCentralDirectory0
was reallocating a new buffer which it failed to free.

This patch rearranges the code a little to ensure that only
buffer gets allocated and it gets freed.

Change-Id: I3b22e2b904f6e7584ab8a3af49c8f1350a40be87

Add a dual clock profiler tracing mode.

Dalvik previously supported using either the thread-cpu clock
or the real time clock as the timebase for profiler traces.
This change adds a dual clock mode where both thread-cpu time
and real time timestamps are collected.

Using dual clock mode significantly improves TraceView ability
to accurately reconstruct the global timeline of events,
particularly on SMP systems.

For now, thread-cpu mode remains the default.

Dual clock mode can be enabled by running the following command
and restarting the system server.
   adb shell setprop dalvik.vm.extra-opts -Xprofile:dualclock

Change-Id: I14db2ae93325ac01efcc8ed02e8747cc0e834e29

Trim the heap when idle after concurrent GC's.
Bug: 4984352

When applied system-wide, trimming the heap can help recover
multiple megabytes worth of pages that are no longer used.
This is especially useful for applications that periodically
allocate large objects during initialization that they then
discard when running in the background.

Change-Id: Iafffd65a70ae62022f0c74f836a84a7afbe23c3c

Don't use dvmIsValidObject outside the GC.

Use dvmIsHeapPointer outside the GC. (This still isn't safe because there's
no synchronization when dealing with the HeapSource.)

Bug: 5049447
Change-Id: Ie0b325ef0a92687ea1eaf1491a4bb832298893c5

Fix the sense of a conditional in the JNI compatibility mode.

Bug: 4772166
Bug: 5041646
Change-Id: I3fb517972920dd518ae1bbe9e3616cb28bb24d4c

Stop advertising the removed -Xdeadlockpredict option.

Bug: 5038293
Change-Id: Ibb4ee3f68a1e3abababc09a51424c5d1c7b1ea0c

resolved conflicts for merge of 72b7c617 to master

Change-Id: I8c3a32d9ee0dea08cca79031cda870141aacdb17

am 68e8f6c0: am 53015360: am 5a57e766: Merge "Change dx.bat to Windows EOL."

* commit '68e8f6c07d1d875dab74f85dcfe0c0d7ac813e29':
  Change dx.bat to Windows EOL.

am f54c7016: am c8d71801: resolved conflicts for merge of d0d351b2 to gingerbread-plus-aosp

* commit 'f54c7016261fefdf146137d50b63ca91ac21b80e':
  Only use O_BINARY if it is defined (windows compatibility)
  Update to allow x86-atom build for FI on Gingerbread
  Combined patch from master branch

Merge "Tidying of the jweak fix."

Merge "Remove the simulator target from all makefiles. Bug: 5010576"

Tidying of the jweak fix.

Bug: 4260055
Change-Id: I1fd56f6c0ea1e2a7245fb2551daf2099bec99709

Remove the simulator target from all makefiles.
Bug: 5010576

Change-Id: I544b54b673d9fd42197a5c97a291578a0bd0db5f

resolved conflicts for merge of c3ff6a0b to honeycomb-plus-aosp

Change-Id: I6b9cfdc27b2654bd832cfd06678a173449f992bc

am bf62959b: Merge "Dex for Windows: Support paths with spaces."

* commit 'bf62959bd8382e458a1aee6b491cb7da703984fb':
  Dex for Windows: Support paths with spaces.

Merge "Dex for Windows: Support paths with spaces."

Fix the jweak implementation.

We need to distinguish between "cleared weak global" and "deleted weak global".
Previously we used NULL for both. Now we add a magic value for cleared weak
globals. I've also switched the GC over to using iterators, so IndirectRefTable
itself becomes responsible for not showing bad pointers to the GC.

I've also improved the reference table dumping to cope with the new scheme and
to be a bit easier to read (through extra indentation).

Bug: 4260055
Change-Id: I26af301fb2b46d014c6f6b0915a8f8a7fb6d7c5b

am 53015360: am 5a57e766: Merge "Change dx.bat to Windows EOL."

* commit '53015360753df7325e3cc84c44bee5b48b497436':
  Change dx.bat to Windows EOL.

am 5a57e766: Merge "Change dx.bat to Windows EOL."

* commit '5a57e766a662184902e215b52cb943e02d9fa827':
  Change dx.bat to Windows EOL.

Merge "Change dx.bat to Windows EOL."

Dex for Windows: Support paths with spaces.

1- Change the format of the command line parsed by dx.jar.
Parameters with values are usually specified with one
single argument e.g.
   --output=/some/path with spaces
That breaks for a variety of reasons on windows, so the first
step is to allow a new 2-parameter form:
   --output "/some/path with spaces"

2- The next step is fixing the way the dx.bat wrapper
extracts the -J parameters to be able to cope with
arguments that contain spaces.

SDK bugs: 4567, 18388

Change-Id: Ia837e7307998c0bf41ae39abb36abebfc27da104

am c8d71801: resolved conflicts for merge of d0d351b2 to gingerbread-plus-aosp

* commit 'c8d718012b0ac4abf668a9f494c33d2dd8a314e5':
  Only use O_BINARY if it is defined (windows compatibility)
  Update to allow x86-atom build for FI on Gingerbread
  Combined patch from master branch

Change dx.bat to Windows EOL.

Change-Id: I0adc6101297e8491814599d345b3cfd3278837cd

resolved conflicts for merge of d0d351b2 to gingerbread-plus-aosp

Change-Id: I1594c21fb3dd761b1e2275be2019ec139cf64174

Add JNI app bug workarounds.

Specifically, this hands out direct pointers for all local references,
and lets you use a JNIEnv* on the wrong thread. This is off by default,
but enabled for apps that don't have ICS as their targetSdkVersion.

Bug: 4772166
Change-Id: I20c403a8e63481a35d579d2bd3b121c80ec08f89

Merge "Allow an app's targetSdkVersion to be set by the zygote."

Allow an app's targetSdkVersion to be set by the zygote.

Bug: 4772166
Change-Id: I7dbfdc2f2856efe608f0474d83326e6f0ba17658

merge intel x86 patches into gingerbread branch

Remove dead code/unused variables to avoid gcc-4.6 warnings.

Change-Id: I291fd42e91085c51772f560d424334874bef8add

Improve a comment.

Change-Id: If6636879dcdc15a33a083a19284de5fe8056e797

Turn on third-party JNI logging if the relevant bit is set.

Change-Id: I3260233516fa2d57c4876056168d31881d8666a9

Don't abort when a weak global's referent is cleared.

This also makes us less likely to output spurious warnings when
dealing with nulled-out weak globals, and lets us provide more
helpful warnings when warnings are called for.

Bug: 4991942
Change-Id: I99b88e66e07f79562da2cd9d594b93bff218d595

Added mutex for jdwp socket writes to prevent interleaving.

This commit includes a following change: Clean up jdwp adb/socket
and fix initialization issues.

Change-Id: I79461b92d690e42d8d1d82d31fbcacb4d5650a74

Fix native method logging to show local references rather than direct pointers.

This is necessary (but not sufficient) for debugging third-party JNI bugs.
It's the second half of the logging story, but still doesn't address the
question of "how does the developer turn on the logging?".

This removes the variant JNI bridges at the cost of adding a couple of
booleans to struct Method. Performance is about the same, except synchronized
native methods are quite a bit faster after the change.

Before:
                             benchmark  ns linear runtime
                      _emptyJniMethod0 333 ==========
                      _emptyJniMethod6 367 ===========
                     _emptyJniMethod6L 921 ==============================
                _emptyJniStaticMethod0 259 ========
                _emptyJniStaticMethod6 287 =========
               _emptyJniStaticMethod6L 873 ============================
    _emptyJniStaticSynchronizedMethod0 404 =============
          _emptyJniSynchronizedMethod0 452 ==============

After:
                             benchmark  ns linear runtime
                      _emptyJniMethod0 344 ==========
                      _emptyJniMethod6 348 ==========
                     _emptyJniMethod6L 969 ==============================
                _emptyJniStaticMethod0 265 ========
                _emptyJniStaticMethod6 293 =========
               _emptyJniStaticMethod6L 968 =============================
    _emptyJniStaticSynchronizedMethod0 265 ========
          _emptyJniSynchronizedMethod0 323 ==========

A better optimization for the case where there are reference arguments
would be to keep a list of argument indexes in the struct Method, so we
could iterate directly over those arguments that need converting to
local references. That would also let us do something about the overhead
of repeatedly looking up which local reference table and cookie to use.

But now is not the time.

Change-Id: Ie32daca1b31be057a44f1ed4b5d28d1634380e1d

Merge "Add two new JNI debugging options."

Add two new JNI debugging options.

The alwaysCheckThread option provides some backwards compatibility for apps
that misuse JNIEnv*s across threads. The logThirdPartyJni is a step towards
making it easier for third-party developers to debug their JNI errors.

Change-Id: I134374da0fe94f3fbc6b6d5aef52e3eef658aff9