Merge "Add a regression test for code.google.com/p/android/issues/detail?id=7935." into dalvik-dev

Merge "Filling out implementations of java.util." into dalvik-dev

Filling out implementations of java.util.

The new code comes straight from Harmony. None of the below classes
were divergent from Harmony so the change was quite straightforward.

The changes have been tested against Harmony's test suite and jtreg.
I haven't added any new tests to our suite, but I don't need to.

Add a regression test for code.google.com/p/android/issues/detail?id=7935.

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

Add a regression test for another DST-less locale.

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

Merge "Remove almost all of the temporary allocations." into dalvik-dev

am cfb71ee5: (-s ours) no emma instrumentation on core-test

Merge commit 'cfb71ee5d72b7514d05b782c780d20cc5e6f73cb' into dalvik-dev

* commit 'cfb71ee5d72b7514d05b782c780d20cc5e6f73cb':
  no emma instrumentation on core-test

Remove almost all of the temporary allocations.

The main source of garbage when running the tests is now
OutputStream.write(int)'s creation of temporary single-byte byte arrays, but
no performance-conscious caller should be using that. In sensible cases, we
now produce very little garbage.

Change-Id: I13b5c8aef9a48c6aca33b039af2b13d5613a5f05

no emma instrumentation on core-test

* applying emma instrumentation causes OOM in dx
* test packages don't need emma instrumentation anyway, it's the
  package under test that needs to be processed by emma

Change-Id: I8e65d3750aae716c52fb98d0b8cd41a748344516

Merge "Fix HttpURLConnection.setFixedLengthStreamingMode." into dalvik-dev

Fix HttpURLConnection.setFixedLengthStreamingMode.

Add better testing of chunked and fixed-length modes, and fix fixed-length mode
so it doesn't try to buffer all the output (which is the last thing you want
because the only point of that mode is to explicitly disable as much buffering
as possible).

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

merge more modules into luni

Merge "Optimize idiv-int/lit for powers of 2." into dalvik-dev

Restore test expectations inadvertently deleted with vogar.

Optimize idiv-int/lit for powers of 2.

before:

                   DivideIntByConstant10  32 XXXXXXXXXXXXXXXXXXXXXXXXXXXX|
                  DivideIntByConstant100  32 XXXXXXXXXXXXXXXXXXXXXXXXXXXX|
    DivideIntByConstant100_HandOptimized  34 XXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
                    DivideIntByConstant2  32 XXXXXXXXXXXXXXXXXXXXXXXXXXXX|
                 DivideIntByConstant2048  22 XXXXXXXXXXXXXXXXXXX|||||||
                    DivideIntByConstant8  20 XXXXXXXXXXXXXXXXX||||||||
                   DivideIntByVariable10  21 XXXXXXXXXXXXXXXXXX|||||||
                    DivideIntByVariable2  21 XXXXXXXXXXXXXXXXXX|||||||

after:

                               benchmark  ns logarithmic runtime
                   DivideIntByConstant10  32 XXXXXXXXXXXXXXXXXXXXXXXXXXXXX
                  DivideIntByConstant100  32 XXXXXXXXXXXXXXXXXXXXXXXXXXXXX
    DivideIntByConstant100_HandOptimized  33 XXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
                    DivideIntByConstant2  11 XXXXXXXXX|||||||||||
                 DivideIntByConstant2048  13 XXXXXXXXXXX|||||||||||
                    DivideIntByConstant8  13 XXXXXXXXXXX|||||||||||
                   DivideIntByVariable10  21 XXXXXXXXXXXXXXXXXXX|||||||
                    DivideIntByVariable2  22 XXXXXXXXXXXXXXXXXXXX||||||

Bug: 2614702
Change-Id: I9dde73d80580446a362cdcc9b82959a4b6bfb384

Merge "Fix =/== bug." into dalvik-dev

Fix =/== bug.

Change-Id: I42c1ede0289334b7837d468763fecc61fc4a8bea

Merge "More merging of nio into luni" into dalvik-dev

Removing vogar from Dalvik's git tree; the source is now on code.google.com.

http://code.google.com/p/vogar/

am b59dd513: am 5793d439: am 3e557951: am 42452494: Fix dexdump to build in Windows SDK under Linux

Merge commit 'b59dd51399f17f60c5e2c9909331887417b0778c' into dalvik-dev

* commit 'b59dd51399f17f60c5e2c9909331887417b0778c':
  Fix dexdump to build in Windows SDK under Linux

am 5793d439: am 3e557951: am 42452494: Fix dexdump to build in Windows SDK under Linux

am 3e557951: am 42452494: Fix dexdump to build in Windows SDK under Linux

Merge commit '3e557951fa772390551d50c85ddd584e4370fb15' into kraken

* commit '3e557951fa772390551d50c85ddd584e4370fb15':
  Fix dexdump to build in Windows SDK under Linux

am 42452494: Fix dexdump to build in Windows SDK under Linux

Merge commit '4245249472f7172de80874190559f6d2ef9c3803' into froyo-plus-aosp

* commit '4245249472f7172de80874190559f6d2ef9c3803':
  Fix dexdump to build in Windows SDK under Linux

Merge "Dead code/cruft removal in the collation code." into dalvik-dev

Dead code/cruft removal in the collation code.

I've also renamed ScopedUtfChars::data to ScopedUtfChars::c_str, by analogy
with std::string (since this method has always been more like c_str than data).

This also fixes a few leaks on error paths.

The old code used to go all the way into native code to return a constant hash
code of 1, so I've removed all that and switched to the idiomatic bogo hash
code (with the idiomatic comment).

Change-Id: I25da8c422155860b5ab348786d369c6c7598135c

More merging of nio into luni

Change-Id: Ib4356b3cd57490df8ff246ea6a4bc1c269ba848b

Fix dexdump to build in Windows SDK under Linux

(Merged from master Change I0e57c81d)

am 29af77bf: am ca88104b: am 770379e1: am 6efd4463: Fix a deadlock in the breakpoint code.

Merge commit '29af77bfa00ac560680dea99cf0893fccc257176' into dalvik-dev

* commit '29af77bfa00ac560680dea99cf0893fccc257176':
  Fix a deadlock in the breakpoint code.

am ca88104b: am 770379e1: am 6efd4463: Fix a deadlock in the breakpoint code.

am 770379e1: am 6efd4463: Fix a deadlock in the breakpoint code.

Merge commit '770379e17e694ec6e08f1edb690a5e33ab81c684' into kraken

* commit '770379e17e694ec6e08f1edb690a5e33ab81c684':
  Fix a deadlock in the breakpoint code.

Merge "Import the copying collector." into dalvik-dev

Import the copying collector.

Change-Id: I48ef17afd9e5e8453e560a96d307f4f949d4ab5e

am 6efd4463: Fix a deadlock in the breakpoint code.

Merge commit '6efd446399ba7a9bb68bf8ddb5a458072a1381b0' into froyo-plus-aosp

* commit '6efd446399ba7a9bb68bf8ddb5a458072a1381b0':
  Fix a deadlock in the breakpoint code.

Fix a deadlock in the breakpoint code.

In froyo we started using "hard" breakpoints, where we replace the
existing opcodes with breakpoint instructions.  This requires some
coordination to avoid confusing the verifier.  The previous approach
allowed the breakpoints to be inserted, and "undid" them while the
verifier ran; this worked, but caused us to be holding a lock for
an extended period.

The new approach just avoids altering the bytecode of unverified
classes, and then "flushes" the breakpoint set out between the time
when verification completes and class initialization starts.  This
removes the possibility of blocking with the lock held, and makes
everything much simpler.

For bug 2615063.

(cherry-pick from dalvik-dev)

Change-Id: I75f19b0cc71fc0babb50ab299c6c5a865e06c919

Merge "Fix a deadlock in the breakpoint code." into dalvik-dev

Merge "Fixing some logging tests, see b/issue?id=2487338" into dalvik-dev

Fix a deadlock in the breakpoint code.

In froyo we started using "hard" breakpoints, where we replace the
existing opcodes with breakpoint instructions.  This requires some
coordination to avoid confusing the verifier.  The previous approach
allowed the breakpoints to be inserted, and "undid" them while the
verifier ran; this worked, but caused us to be holding a lock for
an extended period.

The new approach just avoids altering the bytecode of unverified
classes, and then "flushes" the breakpoint set out between the time
when verification completes and class initialization starts.  This
removes the possibility of blocking with the lock held, and makes
everything much simpler.

For bug 2615063.

Change-Id: I7f43e09a755fba27b335454659b3f04e8b2179ac

Fixing some logging tests, see b/issue?id=2487338

Remove unnecessary IPv6 and IPv4 parsing from URI.

Cleanup URI and InetAddress exception localization.

Merge "Fixing java.lang.NullPointerException at java.net.URI$Helper.isValidDomainName" into dalvik-dev

Fixing java.lang.NullPointerException at java.net.URI$Helper.isValidDomainName

See http://b/issue?id=2604061

Merge "java.text.RuleBasedCollator fixes." into dalvik-dev

java.text.RuleBasedCollator fixes.

Add expectations for broken harmony tests, add our own equivalent (but correct)
tets, and fix the bug turned up by the correct tests: the icu4jni
RuleBasedCollator was using toString to convert a CharacterIterator to a
String, resulting in iteration over the result of Object.toString (the class
name and identity hash code) rather than the characters of interest.

Also shut javac up about non-ASCII characters in Locale.java.

Bug: 2608742
Bug: 2608750
Change-Id: I2171789058c8116eacd7e5815bd483f0bc07c69b

am ed4035bc: am d2203aa0: am 1beb43af: merge from open-source master

Merge commit 'ed4035bca3233c7a40c4d5679c3f7dfea9516c7e' into dalvik-dev

* commit 'ed4035bca3233c7a40c4d5679c3f7dfea9516c7e':
  Improve the implementation of countOnes function to use only 12 operations.

am d2203aa0: am 1beb43af: merge from open-source master

am 1beb43af: merge from open-source master

Merge commit '1beb43af9350d47016c0e3ec55c1b2d4230cae7e' into kraken

* commit '1beb43af9350d47016c0e3ec55c1b2d4230cae7e':
  Improve the implementation of countOnes function to use only 12 operations.

merge from open-source master

Change-Id: Ia08d4f55f5cdced2878777e0812c8cbbf26bb0d6

Update to the latest upstream collator tests.

I've pulled out the not-obviously-insane stuff we added, though I don't
know how useful it is. This change is mainly about reverting our broken
changes to these tests.

Bug: 2608750
Bug: 2608742
Change-Id: Ia4d0a7b12bfc5dfc3fad4b72254918acf74b418d

Merge "Suppress all failures in setSecurityManager() with vogar." into dalvik-dev

Suppress all failures in setSecurityManager() with vogar.

am 3825d34f: am d4b6f6aa: am 1dd42bb9: Fix NetworkInterface on sholes/ppp.

Merge commit '3825d34f9cd5057504ced60962638ffb54f14e38' into dalvik-dev

* commit '3825d34f9cd5057504ced60962638ffb54f14e38':
  Fix NetworkInterface on sholes/ppp.

am d4b6f6aa: am 1dd42bb9: Fix NetworkInterface on sholes/ppp.

am 1dd42bb9: Fix NetworkInterface on sholes/ppp.

Merge commit '1dd42bb9da388c55191449b848f9d82dc4673229' into kraken

* commit '1dd42bb9da388c55191449b848f9d82dc4673229':
  Fix NetworkInterface on sholes/ppp.

Merge "Fix Locale.getDisplayName for unlocalizable languages/countries/variants." into dalvik-dev

Fix NetworkInterface on sholes/ppp.

We need to use IFA_LOCAL to get the interface addresses; IFA_ADDRESS is the
remote address (which is usually the same as IFA_LOCAL, but not in the ppp
case).

Tested on sholes, where it fixes the test used in the bug; also tested on
passion, where it doesn't break the already-working test.

Bug: 2575983
Change-Id: Ib334a002dc099daeeb05cbc341fe233d4d11d2f7

Merge "Add some .odex info to dexdump output." into dalvik-dev

Fix Locale.getDisplayName for unlocalizable languages/countries/variants.

This does not address the interpretation of script identifiers, but does bring us
in line with the RI's behavior. That's why the test doesn't use the example from
the bug.

Bug: 2611311
Change-Id: I54af89aaf418cd520dc6ebdb1e27dc1ac373f70f

Rewrite the HTTP connection pool used by HttpURLConnection.

This started off as incremental changes, but it ended up going
far enough that it earned the "rewrite" badge.

System.getProperty() is not called for every HTTP connection.
This is slightly controversial, but the old code had to bend
over backwards to support dynamic pool changes, and it didn't
even support the case when the pool shrank but not to 0.

The new code doesn't do I/O within static synchronized blocks.
This should reduce contention described here: http://b/issue?id=2606547

Also: simpify implementation, prettier names, more focused doc.

am 0691c470: am 6f7ec8e6: am 28f4d74b: Merge "Remove dangerous logging code from unlockMonitor.  It is unsafe to dereference a potentially corrupt lock word to retrieve additional information about the lock state.  This code was added to debug a transient error and ha

Merge commit '0691c4701cf8cd32db1bc034dc5f544a807aeb3d' into dalvik-dev

* commit '0691c4701cf8cd32db1bc034dc5f544a807aeb3d':
  Remove dangerous logging code from unlockMonitor.  It is unsafe to

am 7e3d5fcd: Merge "include emma into core library on conditional flag"

Merge commit '7e3d5fcddef15bf2dff71c2fe5ee794536819973' into dalvik-dev

* commit '7e3d5fcddef15bf2dff71c2fe5ee794536819973':
  include emma into core library on conditional flag

am 91be9e67: am 9cfe27bd: am 8068ea8d: Update the ssl test so the client reads something from the server. This is needed when cut-through feature is needed as in b/2586347. Dr. No approved in b/issue?id=2511073 .

Merge commit '91be9e671d7db5444aa8fa5f0101867ba8a73075' into dalvik-dev

* commit '91be9e671d7db5444aa8fa5f0101867ba8a73075':
  Update the ssl test so the client reads something from the server.

Fixing TimeZone.setDefault() documentation.

Set expectations for date formatting to tolerate "GMT-07" in the
output. The alternative is to emit an ambigous date string.
Applications that want pretty date formatting should use the
proper facilities: SimpleDateFormat, etc.

This problem was detected by org.apache.harmony.luni.tests.java.util.DateTest.test_toString

am 6f7ec8e6: am 28f4d74b: Merge "Remove dangerous logging code from unlockMonitor.  It is unsafe to dereference a potentially corrupt lock word to retrieve additional information about the lock state.  This code was added to debug a transient error and has since been

am 28f4d74b: Merge "Remove dangerous logging code from unlockMonitor.  It is unsafe to dereference a potentially corrupt lock word to retrieve additional information about the lock state.  This code was added to debug a transient error and has since been removed from

Merge commit '28f4d74b59e9e40278c4f1fe797d8c75f51b5326' into kraken

* commit '28f4d74b59e9e40278c4f1fe797d8c75f51b5326':
  Remove dangerous logging code from unlockMonitor.  It is unsafe to

Merge "Remove dangerous logging code from unlockMonitor.  It is unsafe to dereference a potentially corrupt lock word to retrieve additional information about the lock state.  This code was added to debug a transient error and has since been removed from other branches." into froyo

resolved conflicts for merge of ee25fc06 to dalvik-dev

Change-Id: Ib8c3c62b3b791a8087d7e6298006a6221726cf78

Merge "Fixing reset() on SAXParser and DocumentBuilder, and cleaning up nearby tests." into dalvik-dev

Add some .odex info to dexdump output.

"dexdump -f" will now show the contents of the optimized DEX header if
one is present.  A "table of contents" for the auxillary data is also
displayed.

As an added bonus, stop being lazy and sanitize the DEX magic before
printing it (it has an embedded newline and \0).

Change-Id: If9c5278785062cd0f44fc3d3ea0725ef0009932a

Merge "include emma into core library on conditional flag"

Fixing reset() on SAXParser and DocumentBuilder, and cleaning up nearby tests.

Also fixing vogar to support a directory of test expectations, so we
can group them by file rather than colocating failures we want to fix
with those that we don't.

Merge "Fix a BigInteger(String) bug I introduced yesterday." into dalvik-dev

Merge "Make plain "-Xint" work." into dalvik-dev

Fix a BigInteger(String) bug I introduced yesterday.

I broke parsing of integers in bases greater than 10.

Change-Id: I94c4a75c613a1cd8105b7477e68275a81d04d01b

Make plain "-Xint" work.

Change-Id: I7a09a886df9722b60a861969f3e1a1a45988068a

Improve the implementation of countOnes function to use only 12 operations.

Change-Id: I01b62606a0c87b2937572f8cb7beafc956867353

am 9cfe27bd: am 8068ea8d: Update the ssl test so the client reads something from the server. This is needed when cut-through feature is needed as in b/2586347. Dr. No approved in b/issue?id=2511073 .

am 8068ea8d: Update the ssl test so the client reads something from the server. This is needed when cut-through feature is needed as in b/2586347. Dr. No approved in b/issue?id=2511073 .

Merge commit '8068ea8daa37ac1e4d55c2ebd15bf9dda3d7891e' into kraken

* commit '8068ea8daa37ac1e4d55c2ebd15bf9dda3d7891e':
  Update the ssl test so the client reads something from the server.

Update the ssl test so the client reads something from the server.
This is needed when cut-through feature is needed as in b/2586347.
Dr. No approved in http://b/issue?id=2511073 .

Change-Id: Id84724873522fe0435dbda342616da02783f7d6b

Merge "Merge commit 'a0a3ad10' into manualmerge" into dalvik-dev

Remove dangerous logging code from unlockMonitor.  It is unsafe to
dereference a potentially corrupt lock word to retrieve additional
information about the lock state.  This code was added to debug a
transient error and has since been removed from other branches.

Change-Id: I5aabae531c52532929e8dbb71bdcc6cb7cee64f9

am 831eec93: am b8fcf57f: Fix remaining small issues with the lock sampling.

am b8fcf57f: Fix remaining small issues with the lock sampling.

Merge commit 'b8fcf57f13b4d37950cfbd72a6af401941d7bdd8' into kraken

* commit 'b8fcf57f13b4d37950cfbd72a6af401941d7bdd8':
  Fix remaining small issues with the lock sampling.

Since we no longer make use of formatted I/O in the unlock monitor
exception case, revert to using dvmThrowException which does not treat
the description string as an output format.

Change-Id: Ice1dc04cb6d1e887af718022cc0b03c6d90f9a19

Fix remaining small issues with the lock sampling.

* Eliminate the now unused lockprofsample flag.  The sample percentage
  has been a function of the wait time an threshold since the previous
  change.

* Eliminate some trailing whitespace that creeped into the previous
  change.

* Eliminate the trailing newline from the event buffer.  Because the
  buffer pointer was not incremented this character was not seen in
  the event log anyway.

* Document the lockProfThreshold global.

Change-Id: Ia1c1fcf862d76f5631601e05e7941e5171fab097

Merge commit 'a0a3ad10' into manualmerge

Conflicts:
libcore/x-net/src/main/native/org_apache_harmony_xnet_provider_jsse_NativeCrypto.cpp

Change-Id: I23508e348c5de4103e9dda619ca230d0575f57ca

am 5005a521: am 612fa566: am af69cf8d: Fix several issues with the lock sampling code.  This change obsoletes the lockProfSample flag.

Merge commit '5005a5210b941d18b7742469ba622b1c8a754b1e' into dalvik-dev

* commit '5005a5210b941d18b7742469ba622b1c8a754b1e':
  Fix several issues with the lock sampling code.  This change obsoletes

am bd4eaf95: am 54c9323e: am 6b4ba58e: Add a mechanism to long a sample of long running lock acquisitions.

Merge commit 'bd4eaf959e21f3e048366141896ce262f22e0e2c' into dalvik-dev

* commit 'bd4eaf959e21f3e048366141896ce262f22e0e2c':
  Add a mechanism to long a sample of long running lock acquisitions.

am a731357c: (-s ours) am a8984fe8: (-s ours) am ddc0d4c2: (-s ours) DO NOT MERGE. Suppressing tests that fail in CTS\' test runner.

Merge commit 'a731357c0cadaca3863f6fd0de1b8ffb39464220' into dalvik-dev

* commit 'a731357c0cadaca3863f6fd0de1b8ffb39464220':
  DO NOT MERGE. Suppressing tests that fail in CTS' test runner.

Fix U_FILE_ACCESS_ERROR RuntimeException in String.getBytes("GB18030").

Bug: 2606807
Change-Id: Ie639ce6380e42e0eed2a7e360504f86a22f1fed8

Fix several BigInteger failures.

From Java 7 on, BigInteger also accepts a leading "+" in its string
constructors.

BigInteger has always claimed to accept non-ASCII digits, but our
implementation never has.

BigInteger.isProbablePrime is defined to return true for certainty <= 0,
but OpenSSL (on which we're based) takes the opposite stance.

Change-Id: I00bfc591a4d583460f71b7eec3de91bf6b03cd87

Merge "Speed up %d for locales with non-ASCII digits." into dalvik-dev

Speed up %d for locales with non-ASCII digits.

The small "localizeDigits" part of this change is the big speedup. It takes
%d in the "ar" Locale from 5x more expensive than "en_US", to just a small
percentage more expensive.

The big part of this change: removing the useless Transformer and FloatUtils
classes touches many lines (mostly just changing the level of indentation),
and only has a small percentage effect on performance. The motivation for
this is actually primarily that it makes the code easier to read, and makes
it easier to share code.

Change-Id: I07e86a3ee448510d9ae36579a9af283349f777f1

Merge "openssl-1.0.0 upgrade"

Tidying up the error reporting.

We were spamming our logs with a bunch of redundant messages,
particularly surrounding timeouts. This should reduce the log
output to only the useful stuff when a test takes too long
to complete.

Also not blowing away the full XML report when a few tests
get lost along the way.

Remove a dangling import, and another unused Messages.java.

Also tidy up our two awt.font classes.

Change-Id: I931d348eef7265190f76c31f74c9a5d9ec5d6597

Merge "Remove the nio Messages.java I missed, and the three now-unused imports." into dalvik-dev

Remove the nio Messages.java I missed, and the three now-unused imports.

I seem to be having trouble deleting files lately...

Change-Id: I019acd19aa61acdeeee1ab6f7fb8a063a41a5534

Small vogar changes in response to the review for change 49052.

Merge "Teaching vogar to run prebuilt .jar files." into dalvik-dev

Merge "Remove "messages" from the logging, math, and nio_char modules." into dalvik-dev