Don't leak Repository and RevWalk instances during notedb parsing

Change-Id: Ia8d99f0387206295678234a549dc22a062e5655b
Merge "Update Buck"
Update Buck

Buck no longer accepts duplicate deps. One test listed a library
that is always included for tests and can be safely dropped.

Buck does not accept duplicate resources, which happens with
gwt_module() loading in srcs, resources and gwt_xml. Convert
to a set and back to a list to remove duplicates transparently.

Change-Id: I0e450ef376040e8fa315ca17ab62a5cebbaf1d67
ReplyBox: Simplify ui:msg wrapping on Post

Since 00aa3bfb977faf6e removed the "send email" checkbox from
the ReplyBox UI the <ui:msg> no longer had to include the button.
It is sufficient to wrap only the message text, similar to Cancel.

Change-Id: I3da5dee34187071b76370f1112d7da7b2c50534c
Add missing private constructor for static utility class LoginUrlToken

Change-Id: Ia9f81295de7542b6b090d378456fefc95cf6e891
Merge changes Ia9f81295,I3da5dee3

* changes:
  Add missing private constructor for static utility class LoginUrlToken
  ReplyBox: Simplify ui:msg wrapping on Post
Simplify BanCommitResult type

The default constructor is not required, as Java will supply one.

Use ArrayList(4) instead of LinkedList. Its about the same size
for 1 commit, and smaller for higher numbers. Callers do not
perform insertions or removals from the middle of the lists.

Change-Id: Ib0e0c857697a6347dcb90c4c14f84555c2fd7c2a
Update replication plugin to use correct ref names in tests

Change-Id: I667fcc9d9de6e8b2e20fe105ec83f7141b57b850