Set hasValue readOnly. Fixes #393

[manolo]

This change is 

[platosha]

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

---

test/vaadin-combo-box-properties.html, line 89 at r1 (raw file):

      describe('hasValue property', function() {
        it('should be readOnly', function() {
          expect(comboBox.hasValue).to.be.false;

Should we test <vaadin-combo-box-light> as well?

---

Comments from Reviewable

[manolo]

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

---

test/vaadin-combo-box-properties.html, line 89 at r1 (raw file):

Previously, platosha (Anton Platonov) wrote…

Should we test <vaadin-combo-box-light> as well?

Well, changes are in ComboBoxBehavior which is implemented by both.
Since what we are testing is the implementation, using any element must work to be sure that the code is correct.
I would test in both if the code is repeated, or there was some kind of overriding of the behaviour.

---

Comments from Reviewable

[platosha]

---

Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

---

Comments from Reviewable

[limonte]

---

Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

---

Comments from Reviewable