Trivial cleanups:
- Remove stray paren in Ordering Javadoc.
- Fix runnable/executor confusion in ExecutionQueue Javadoc.
- Fix MockingModuleTester and FastStackTracer links that refer to type parameter T instead of its valid name, the erased type Object. Fix a few @see errors while there.
- Remove "This class is GWT compatible" comments, which have been redundant since we began using @GwtCompatible in testlib code.
- "unreasonable slow" => "unreasonably slow"
- Fix continued-line indent in Invokable.
- Insert spaces in cast expressions.
- Add missing spaces before open braces.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=47059317

Add application/protobuf mime type, from
http://tools.ietf.org/html/draft-rfernando-protocol-buffers-00
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=47014458

Delete completely unused ConcurrentMapInterfaceTest.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=47004919

Guava issue 1423: fix TreeMultimap.asMap().headMap to properly register inclusivity argument
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=47004337

Remove tests for Multimap.get from AbstractMultimapTest that duplicate tests in MultimapTestSuiteBuilder
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46990147

Add test suites for Multimap.equals to MultimapTestSuiteBuilder
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46989151

Migrate Multimap.clear() tests (and tests to clear Multimap views) to MultimapClearTester.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46987144

Remove tests from AbstractMultimapTest that overlap with MultimapRemoveTester and MultimapRemoveAllTester in MultimapTestSuiteBuilder.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46986379

Add MapToStringTester and MultimapToStringTester, and fix ForwardingNavigableMap.StandardDescendingMap.toString() to correctly iterate in descending order.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46952541

Remove get().subList() tests from AbstractListMultimapTest that are redundant with ListSubListTester in the derived test suite for ListMultimapTestSuiteBuilder
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46951757

Add explicit type parameters in FilteredMultimapValues to make JDK6 happy.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46951071

Delete Multimap.contains{Key,Value} tests from AbstractMultimapTest that are redundant with MultimapContains{Key,Value}Tester.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46951020

Remove tests from AbstractListMultimapTest that are redundant with ListMultimapPutTester.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46950561

Delete tests from AbstractMultimapTest of the multimap views in isolation, that is, tests that initialize the multimap, get the view, and don't reference the multimap afterwards.

These tests are redundant with the derived TestSuiteBuilders in MultimapTestSuiteBuilder.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46949401

Migrate all remaining Multimap.replaceValues tests from AbstractMultimapTest to MultimapReplaceAllTester
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46948091

Add tester to MultimapTestSuiteBuilder for Multimap.putAll(Multimap)
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46946989

Replace Multimap.putAll(K, Iterable) tests from AbstractMultimapTest with more exhaustive tests in MultimapPutIterableTester
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46945862

Add the TestSuiteBuilder for Multimap.values(), and fix Multimaps.filterXXX to pass those tests properly.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46945446

Remove tests of Multimap.get() from AbstractMultimapTest that are duplicated in MultimapGetTester
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46942143

Delete tests from AbstractMultimapTest that duplicate tests in MultimapSizeTester.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46940075

Add Multimaps.asMap() helpers that return a Map with more
strongly-typed values than ListMultimap#asMap(), etc.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46939830

Add tests to the asMap() views of ListMultimap, SetMultimap, and SortedSetMultimap to verify that the asMap().values() are of the correct type (List, Set, SortedSet) at runtime.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46939057

Change return type of openBufferedStream() on ByteSource, ByteSink and CharSink to the base stream type rather than the Buffered* type to allow implementations to avoid unnecessarily adding additional buffering to streams that do not benefit from it.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46926342

Reshuffling from internal-only changes.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46885542

Shuffling from internal-only changes.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46844311

Give removal dates for Stopwatch ctors.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46791249

Improve error messaging in EqualsTester.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46753546

Implement Object#equals() and #hashCode() for stateful HashFunctions.
Add tests to verify the behavior.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46751376

Override ImmutableBiMap.containsValue() to use the O(1) implementation, as per Guava bug 1416.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46744049

Give removal dates (releases) for HashFunction#hashString(CS) and
Hasher#putString(CS).
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46638784

Add reference to Funnels from Funnel javadocs.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46629147

Change SynchronizedEventHandler to use a synchronized block rather than just synchronizing the method. Since the method body does nothing but call the super method, it (and its synchronization) was apparently being optimized away under a certain JVM: https://code.google.com/p/guava-libraries/issues/detail?id=1403
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46627764

Use com.google.caliper.legacy.Benchmark instead of com.google.caliper.Benchmark. This CL facilitates the move from a superclass-based API to an annotation-based API. Benchmark and legacy.Benchmark are equivalent classes, so this is NOT a behavior changing CL.

-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46556497

fixed NPE caused by inaccessible directory, as reported by Guava issue 1407.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46478011

Fix the logging in Futures.CombinedFuture such that:

1. Errors are always logged (even if they were put into the output future)
2. If allMustSucceed is true then all failures after the first are logged
3. if allMustSucceed is false then all failures are not logged.

In the prior implementation (except for in rare races) exceptions were only logged if allMustSucceed was false.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46470009

Some whitespace edits

-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46404300

Update Mozilla PSL.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46368166

Add some missing dependencies in a GWT module.

-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46363258

Add Funnels#unencodedCharsFunnel (and deprecate Funnels#stringFunnel()).
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46350462

Remove the rethrow Error behavior from AbstractFuture.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46315153

Add a test for hashInt() vs. a loop w/ putByte().
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46295877

Change ServiceManager.Listener to be an abstract class with empty implementations of its callbacks instead of an interface.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46244288

Guava issue 1322: deal with a potential overflow issue in ImmutableTable.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46160775

Fix typo in Maps.immutableEnumMap. See https://code.google.com/p/guava-libraries/issues/detail?id=1397
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46145330

Change Service.Listener from an interface to an abstract class with no-op implementations of all the callbacks.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46026613

Allow a max size of 0 for EvictingQueue
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=46003229

Use hashFoo() shortcuts instead of newHasher().putFoo().hash().
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45995812

Remove incorrect test using UTF_16LE.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45985871

Changed Invokable.from(constructor).getReturnType() to return ParameterizedType instead of raw class when the class is parameterized. This makes constructor invokable more consistent with factory method invokable.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45912432

Sort and removed unused imports in com/google/common.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45827926

Allow HashCode#padToLong to work with hashs that have <32 bits.
Fix a small typo in Hasher javadocs.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45717814

CHANGE: Make HostAndPort GwtCompatible by getting rid of a regex.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45669298

Change FileByteSource.read() to not unnecessarily copy the byte array in the normal case where the file size doesn't change.

It was incorrectly assuming that if the first read loop filled the original array, the last read() call would have returned -1 if it was done, which is not the case.

Guava bug: https://code.google.com/p/guava-libraries/issues/detail?id=1376
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45616868

Fix apparently harmless but still incorrect references to module name.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45613704

Remove unused HTML dependency.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45608957

Fix @since for EscaperAsserts.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45601740

Add links to the hash function comparisons to the Hashing javadocs.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45527021

Add a test to demonstrate that hashBytes(Ints.toByteArray(i)) ==
hashInt(Integer.reverseBytes(i)).
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45476839

Fixed a bug in rejectTypeVariable() where we failed to account for the owner type's type variables. For example:

abstract class From<K> {
class To<V> {
TypeToken<To<V>> type() {
return new TypeToken<To<V>>(getClass()) {};
}
}

static <K> TypeToken<From<K>.To<?>> cheat() {
return new From<K>().new To<String>() {}.type();
}

TypeToken<From<String>.To<?>> notReallyString = TheClass.<String>cheat();
notReallyString.rejectTypeVariables(); // Should throw
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45444012

Added TypeVisitor to make it easier to navigate type hierarchy, and used it to replace a few code duplications.

TypeVisitor might be able to become something that's releasable for users to check on individual sub-components of a Type. Or, if after learning use cases, all that's needed are hasTypeVariables() and hasWildcards(), we'll just add the higher level APIs.

See Guava issue: https://code.google.com/p/guava-libraries/issues/detail?id=1362
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45443595

Add tests for common aliases of SHA1.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45427688

Remove Platform.clone(), along with various supersource versions.
The last non-test caller was removed in CL 45290173.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45408889

Reshuffling from removal of internal-only method.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45408398

Release base escaper classes, HTML escapers, XML escapers, and URL escapers into Guava.
Keep the source-code escapers internal until we figure out which ones to recommend and what to name them.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45374049

Added code example to AbstractInvocationHandler to suggest users to use Reflection.newProxy().
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45317940

fix some docs in ExecutionQueue
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45311589

add csiro.au as TLD since is not listed on Mozilla PSL
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45307952

Run "normal" GWT tests in public Guava.
We still don't run the suite-builder tests, so the run is incomplete, but at least it's fairly quick:
Tests run: 6667, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 155.218 sec

Among the steps of this process covered by this CL:
- Export the generated GWTTestCase files into Guava.
- Add dependencies in guava-gwt on the guava (and guava-testlib) test sources.
- Treat the GWT portion of guava-testlib as though it is part of the guava-gwt tests (to sidestep the issue that we should really have a separate guava-testlib-gwt artifact analogous to the separate guava-gwt project). (Mainly this means putting its files under guava-gwt/tests.)
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45160153

Use the renamingDecorator to ensure that Services always run in appropriately named threads.

AbstractIdleService and AbstractScheduledService encode the current state in the name so you should see something like

MyService (STARTING|RUNNING|STOPPING)

AbstractExecutionThread service just uses the serviceName() (so it would be "MyService")  I could additionally try to encode the state but it didn't seem worth it.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45135687

Provide ExecutorService decorators that will rename the threads in which its tasks run for the duration of the task.

Open issues:
1. should any of these be public?  Currently I am only planning on using them for some of the Service base classes so package private is fine, but we may want to make them public.
2. I needed/wanted to use WrappingExecutorService but it was marked GoogleInternal, so i removed @GoogleInternal.  Is this ok? or should i use a similar private class? moe-strip public from it?
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45121293

Apply test.include to testlib tests as well as main tests.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45103307

Closer.register supports null closeable
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45058002

Internal refactoring to make it easy to expose a factory method that configures the burst size
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=45016791

Convert GWT-only tests into normal tests.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44698403

Add ByteSource.empty() and CharSource.empty().
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44815909

Add nullWriter(), a Writer that discards everything written to it.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44805209

Mention the use of slice(offset, length) for viewing a range in the array to the ByteSource.wrap javadoc.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44798251

Fall back to countByReading if InputStream available always returns zero.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44796829

remove the queuedListeners lock from both ServiceManager and AbstractService.

The original idea of splitting the queuedListeners queue into one queue per listenerInstance didn't pan out because it didn't actually solve the issue of not holding a lock while submitting the listener to the executor.  So this is a different strategy.

I pulled all the listener execution logic into a new class called ListenerQueue which is analogous to ExecutionList (not sure about the name though...) which cleaned up both AbstractService and ServiceManager.

The approach is two-fold
1. do not remove a listener from the queue of listeners until it has definitely been submitted to its executor
2. ensure that submitting a listener to its executor happens no more than one time (using a lock).

N.B. we are still vulnerable to a weird or buggy executor implementation. If calling execute(runnable) runs for a long time without calling runnable.run then we can still deadlock.  I don't believe we can actually solve this problem without giving up on the ordering guarantees.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44709344

Convert GWT-only tests into normal tests.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44698403

Fix a copy/paste mistake in CharSource javadoc.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44648058

Add isPrimitive(), wrap(), and unwrap() methods to TypeToken
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44620879

Fix typo in javadoc: the name of the method to override is "compare", not "compareTo".
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44580155

A few small hash changes.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44571077

Deprecate the constructors in com.google.common.base.Stopwatch and Stopwatch#elapsedMillis and #elapsedTime.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44568679

Reduce redundant iterator creation in Multimap.putAll(K, Iterable<? extends V>)
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44559924

new Stopwatch(); -> Stopwatch.createUnstarted();
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44556107

Change ByteStreams.asByteSource calls to ByteSource.wrap and deprecate it.
Change CharStreams.asCharSource calls to CharSource.wrap and deprecate it.
Change both deprecated methods to delegate to the replacement method.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44516201

Replace new Stopwatch().start() with Stopwatch.createStarted().
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44515274

Nullability fixes.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44514905

Copy ByteStreams.asByteSource(byte[]) to ByteSource.wrap(byte[]).
Copy CharStreams.asCharSource(String) to CharSource.wrap(CharSequence).

Add package-private CharSequenceReader and tests, for use by the CharSource.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44506727

Add Class-Path cycle detection to avoid StackOverflowError
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44466033

Throws UOE in MutableTypeToInstanceMap Entry#set()
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44465568

Add static factory methods to Stopwatch: createStarted() and
createUnstarted().
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44465501

Fix up some javadoc in TypeToken re: TypeLiteral.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44411479

Some more refactoring of TypeResolver.TypeTable. I was in a hurry. The TypeTable code wasn't in a form I'd LGTM.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44403308

Make TypeResolver public in Guava.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44329762

Fix bug in statistics benchmarks implementation of Knuth algorithms.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44319466

Change Futures.allAsList and .successfulAsList to return an unmodifiable list.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44306369

Hasher#putString(CharSequence) to Hasher#putUnencodedChars(CharSequence)
HashFunction#hashString(CharSequence) to HashFunction#hashUnencodedChars(CharSequence)
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=44304885