Avoid using java.util collections in GWT code

[Legioth]

This reduces the gzipped size of the client engine by around 10% from
48864 to 44808 bytes.

---

This change is 

[vaadin-bot]

SonarQube analysis reported 1 issue

• 1 minor

Watch the comments in this conversation to review them.

[vaadin-bot]

Remove useless curly braces around statement

[denis-anisimov]

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

---

flow-client/src/main/java/com/vaadin/client/DependencyLoader.java, line 163 at r1 (raw file):

        JsMap<String, BiConsumer<String, ResourceLoadListener>> lazyDependencies = JsCollections
                .map();

Is it a replacement for LinkedHashMap ? So it preserves the order ?

---

Comments from Reviewable

[Legioth]

Review status: all files reviewed at latest revision, 2 unresolved discussions.

---

flow-client/src/main/java/com/vaadin/client/DependencyLoader.java, line 163 at r1 (raw file):

Previously, denis-anisimov (Denis) wrote…

        JsMap<String, BiConsumer<String, ResourceLoadListener>> lazyDependencies = JsCollections
                .map();

Is it a replacement for LinkedHashMap ? So it preserves the order ?

A Map object iterates its elements in insertion order according to https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map

---

Comments from Reviewable

[denis-anisimov]

---

Review status: all files reviewed at latest revision, 1 unresolved discussion.

---

Comments from Reviewable

[caalador]

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

---

Comments from Reviewable