Revert "Disable closure compiler (#13778)"

This reverts commit a699c057afc0512b36fa0cbe3f67b83fc2fb7b73.
Revert "Fix Push inserts producing duplicate rows in Table (#13562)"

This reverts commit ce1764fb1bad1b82462d73fbfe293b676c2a9c00.
Reapply "Disable closure compiler (#13778)"

This reverts commit 2a71c0deb62bc576d80b7720312876da5f684e6c.
- HeaderOnSync tests verified to work manually.
Change ColorPickerInputFormatsTest for use Keys.RETURN.

Change-Id: Ic8ad7163d35e0e1c2d44e928cd52ca66a637b7fa
Convert DateFieldReadOnlyTest TB2 -> TB4.

Change-Id: Iab81b1e5992d47c956f9d0440ece98fa546ee927
Revert "Fix table scrolling up on select (#10106)"

This reverts commit 949588fad9300b0c777d2cdbc46dba15680ef809.

Change-Id: I4d9e101a92e0cd21faf9c412dc5971c896125b95
Prevent adding/setting parent component as child (#12720).

Change-Id: I9b7b43f176e88d9edca591001297b94475a31854
Modified Coding Conventions to not have a new line after annotations on
parameters (#13569).

Change-Id: Idc541f298591453c52285a8b1987052caad79e4f
Optimize getAllChildrenIterable() performance. (#13803)

getAllChildrenIterable() is invoked frequently, its used by
attach/detach and markAsDirty.
Because of the implementation detail in CombinedIterator, on every call
all iterators are checked. This means that on every component which has
multiple childs, each next() call will first check the extension
iterator, before advancing in the child iterator.
By having the childs first, this overhead is reduced. If no extensions
are existing (which is quite common) the iterator is not added at all.
(creating an iterator on an empty UnmodifiableCollection is waste of
time and memory)

Change-Id: I23bb91464052ad4282963ec4b5be8a52c6847d4f
Optimizes ConnectorTracker.cleanConnectorMap (#13803)

cleanConnectorMap is a hot method.
This change improves multiple code paths:
1) The global resource handler is no longer get()ed from the session for
   each connector (was in two loops).
2) The ConnectorTracker is no longer get()ed from the UI for each
   connector (was in one loop).
3) values() iterator for ClientConnectors replaces
   keySet() iterator + get(key) call
4) use getUI() from ClientConnector instead manual method.
5) moved removeUnregisteredConnectors to ease inlining and readability.

Change-Id: I25ff5bc32d27d30ecd8d354922ec6e4a02b05921
Properly deserialize push connection (#12235)

The UI.pushConnection field is transient no longer; instead PushConnection
implementations must take care of serialization internally. When a session
is serialized, the client should notice that push was disconnected and
try to reconnect. A deserialized PushConnection should be in a "disconnected"
state so the eventual client reconnection works correctly.

Change-Id: I38cfc5a5cdbd3643311f830f8d580458dcd85c56
Allow using modifier + enter as shorcuts in TextArea (#13811)

Change-Id: I99db3d6280f0066ed1249cb348da7e82381b45c2
Replace use of deprecated DOM.setStyleAttribute method (#13781)

This change is only a brain dead replacement of one specific method. In
some cases, there were some oddities in the surrounding code as well,
but these were left unresolved on purpose.

It should also be noted that DOM.setStyleAttribute just delegates to
element.getStyle().setProperty(), so any null references would have
caused problems already in the existing code.

Change-Id: I340122ac0767af9928076376f76e5bd2c5e19f9f
Fix Push update race condition (#13562)

Change-Id: I50094bc2d236f6dbb02a8b82d6cc9b5f7e4733a5

Conflicts:
	uitest/src/com/vaadin/tests/components/table/ContainerSizeChangeTest.java
Revert "Force recalc of width when the ComboBox style has changed. (#13444)"

This reverts commit 4058b7b063075fed15771eef30cc87b0f58f0cda.

Revert "Treat initial rendering and style changes separately. (#13444)"

This reverts commit b3d683e974f7c3b2269753aec615872b2cddc180.

Revert "Force recalc of width when the ComboBox style has changed  (#13444)"

This reverts commit 09ca9bc8237a14541890360598130c300509a617.

Change-Id: I8c4edb1079be428f5ccb06ff9da396f9ead8501e