Fix sanitization of empty RTA input for Firefox & IE (#11937)

Browsers differ in what they return as the content of a visually empty rich text area (RTA). Accordingly, RTA sanitizes these different values ensuring an empty string is returned to the framework. However, existing sanitization criteria doesn't work for Firefox 74 and Internet Explorer 11.

This fix appends the sanitization criteria of Firefox 74 and IE 11, ensuring an empty string is returned to the framework for a a visually empty RTA.

Closes #10338

Allow setting Upload button caption as HTML, move data to SharedState. (#11940)

Fixes #11810

Prevent upload if no file is selected. (#11939)

Fixes #10419

Fix NPE in FileUploadHandler when use Upload.interruptUpload(). (#11797)

Fixes #11806

Ensure ChangeListener still works after first upload. (#11936)

Fixes #10420

Ensure recalculateColumnWidths works with refreshAll. (#11934)

Column widths shouldn't be calculated between the clearing of cache and
re-populating it, but be delayed until the cache has some content again.
The calculations should only be triggered immediately if no rows are
expected.

Fixes #9996

Updated LayoutTester screenshots for IE and FF. (#11930)

* Updated LayoutTester screenshots for IE and FF.

Fix an issue with frozen column count and unhiding. (#11929)

* Fix an issue with frozen column count and unhiding.

Unhiding logic should take into account that there can be hidden frozen
columns that are not the column currently getting shown.

Add setters to Criterion to fix serialization. (#11926)

Fixes #11909

Tweaked layouttester tests to be more informative. (#11800) (#11813)

Simplify Grid sidebar handling. (#11920)

There is no need to reset the sidebar for every column that is set
hidable within the same roundtrip. Because no other layouting depends on
the sidebar contents, we can safely delay the reset until all columns
have got their hidabiity status updated. As a bonus point, when the
reset itself is delayed, the headers have time to get populated and the
sidebar button's height can be determined immediately at attach. The
reset ensures that attach is called every time a column's hidability is
changed.

Prevent a gap within Grid rows in some resize situations. (#11918)

Fixes #11892

Trigger re-measure after updating ElementResizeListeners. (#11912)

Removing ElementResizeListeners from an element makes it unmeasurable
and clears any saved measured values. Adding the listeners back makes
the element measurable again but doesn't add it to measuring queue.
Measuring needs to happen or any updates to expanded components within a
layout (without changes that would trigger full re-measuring of the
layout itself) lead to broken expand size calculations with any fixed
size elements assumed to have no size.

Fixes #10734

Further tweaks to ComboBox popup positioning. (#11910)

* Further tweaks to ComboBox popup positioning.

- Updated a comment and renamed a private method for better clarity.
- Blocked unnecessary position updates.
- Added a test for #11866 / #11894.

Adding missing setPopupPosition(left, top); (#11902)

* Adding missing setPopupPosition(left, top);

setPopupPosition(left, top); needs to be called in order to top position to be set

Fixes https://github.com/vaadin/framework/issues/11894

Rewrote debouncing of onResize (#11899)

* Rewrote debouncing of onResize

Fixes https://github.com/vaadin/framework/issues/11892

* Fixing typo in variable name

Removing code causing the  trouble (#11898)

* Removing code causing the  trouble

Removing code that was apparently not needed in previous fix and caused regression

Fixes: https://github.com/vaadin/framework/issues/11895
* Added feature to test UI

* Added test case

* Adding missing import

* Fixing test UI

* Enable DateField

Added note to DateField#setRangeStart JavaDoc (#11886)

* Added note to DateField#setRangeStart JavaDoc

* Added additional checking of valid startDate

* Reverted previous commit

Update Chrome version (#11891)

Fix logic of lexicographical comparison of DateField range end (#11885)

* Fix logic of lexicographical comparison of DateField range end

Fix logic of lexicographical comparison of DateField range end with large year numbers ( > 9999)

Fixes #11881

* Added tests

Purpose of dateFieldRangeYearDigitsIncrease() is to test that rangeEnd works correctly on 4 to 5 digits change boundary

Purpose of dateFieldRangeYearBigNumbersPopupOpens() is to test that popup opens also when there is more than 4 digits in year

* Fixing typo

* Add error message

* Set the date of the field in test

* Fixing

Update Navigator.getState() JavaDoc to be more accurate (#11876)

Fixes: https://github.com/vaadin/framework/issues/11875

Don't reset date to current at DateField state (e.g. read-only) updates. (#11879)

- DateFields with month or year resolution should not get their date
reset to current date if the field's state is updated (e.g. by changing
read-only status or adding a range).

Fixes: #11864, #11605

Update to 8.11 snapshot (#11874)

* Update to 8.11-SNAPSHOT

TreeDataProvider: Apply filters to children (#11868)

Fixes: https://github.com/vaadin/framework/issues/9933
Cherry pick from: https://github.com/vaadin/flow/pull/7317

Ensure value change happens before shortcuts in compatibility components (#11871)

Fixes #10854

Making setEditable friendlier (#11869)

* Making setEditable friendlier

The current behavior is not buggy, so this PR is an enhancement

Fixes https://github.com/vaadin/framework/issues/8718

* Improved JavaDoc

Convert DateTimeFieldReadOnlyTest to not rely on screenshots. (#11870)

Added Note about TextField in Header cell (#11858)

Fixes https://github.com/vaadin/framework/issues/9089

Convert DateFieldReadOnlyTest to not rely on screenshots. (#11867)

Removed outdated spacer height update from removal process. (#11861)

* Removed outdated spacer height update from removal process.

Fixes #11856

Added notes in JavaDoc of TEXTFIELD_INLINE_ICON (#11860)

* Added notes in JavaDoc of TEXTFIELD_INLINE_ICON

Fixes https://github.com/vaadin/framework/issues/10752 and https://github.com/vaadin/framework/issues/9089

Fix to RadioButtonGroup readOnly handling. (#11855)

* Fix to RadioButtonGroup readOnly handling.

Updated the connector to use the widget's enabled and readOnly handling
as intended.

Fixes #11843

Added helper methods for setting only height or width full. (#11854)

Added documentation of newly added Binder features (#11845)

Test tweaks (#11851)

* Test tweaks

Less fixed waiting times when there are ways to check that the operation
has been performed. This will hopefully improve the stability a bit.

* Merge branch 'master' into updateTests20191218

Improvements to popup positioning for ComboBox within HorizontalLayout. (#11846)

Expand ratio and spacing can cause ComboBox to miscalculate its own
position while layouting is still ongoing. Popup should not be
repositioned in such circumstances in order to avoid incorrect
intermediate states.

Continues on #11718

Chrome updated to 79 (#11847)

Release notes (#11842)

Test tweaks (#11841)

- Use BrowserStack workaround for proper sendKeys functionality on IE11.
- TreeGridBigDetailsManagerTest: allow two pixel discrepancy in expected
scroll position to accommodate IE.
- TreeWideContent: switched from raw type to type inference.
- ValoColorPickerInputFormatsTest: switched equals comparison to correct
way around.
- ResponsiveLayoutUpdateTest: updated screenshots for IE
- ColumnCollapsingAndColumnExpansionTest: updated screenshots for IE
- ClosingWindowWithBrowserFrameShouldntGenerate404Test: test is
incompatible with IE driver, excluded from tested browsers. In manual
testing IE doesn't get 404 in the logs with or without the fix
associated with this test but there are other client-side exceptions
related to focus handling in either case. However, those are out of
scope for this particular fix and should be investigated separately.

Close window on ESC, when maximized button is clicked (#11840)

Fixes #11838

Changes:

1. Close a window when maximized button is focused and ESC is pressed
2. Add additional check for a close button to react to the ESC key press
3. Rename a private method `onCloseClick` to `closeWindow` to allow code re-use

Added @since tags, some tweaks to formatting. (#11839)

Make VaadinService.cleanupSession public (#11738)

to allow for better integration of third party applications handling the destruction of the session.

Usage example (see https://vaadin.com/directory/component/cleanupservlet-add-on/overview)

"It's possible to close a browser window in such way that neither UI cleanup nor session cleanup will happen until the underlying http session timeouts. This can happen because the design idea for heartbeat is to keep the UI alive, not to ensure timely cleanup, and as such the default check is only performed at the end of each request."

Simplify Grid scroll handling. (#11835)

If first attempt at scrolling doesn't succeed it's unlikely that
continuing to wait is going to make any difference. Cache should be
populated before triggering any actions that depend on the row being
visible, otherwise it should be enough to trust that scrollToRow
actually scrolls to row and once scrolling is done the row is as much in
view as it's going to get. This way we don't get into a situation where
Editor never opens because it's still waiting for that one last pixel
that can't be achieved thanks to browser zoom causing rounding errors.

Continues on #11672

Make asRequired conditional on binding.setAsRequiredEnabled(..) (#11834)

It is a very common use case in complex form that whether a field is required or not, it depends on input on other fields. Hypothetical use case sample could be that we have form for a Product and price of the product is needed except in case the Product's type is Sample. So in that kind of scenarios it would be needed to turn off asRequired() validation easily. The purpose of this enhancement and new binding.setAsRequiredEnabled(..) API is to help implementation of this kind of use cases more easily.

https://github.com/vaadin/framework/issues/10709

Add method writeBeanAsDraft(bean) in Binder (#11833)

* Add method writeBeanAsDraft(bean) in Binder

With current Binder implementation it is not easy to support Forms, which you want to save as draft, i.e. incomplete. For example there can be big text areas, that require time to fill, or lot of fields. Therefore it is needed to that form can be saved, e.g. to other bean in incomplete state when it is not yet passing validation and this other bean can be persisted to draft storage for further editing in the future. This method helps to achieve that easily.

* Add test case for Binder.writeBeanAsDraft(bean)

Bind a field with validator, set value that does not pass validator and save, assert that value was saved.

* Updating test

* Fixing logic flaw in test

* Further improvement of the test case

* Clarification of the JavaDoc

* Fixing typo

* JavaDoc language check

* Fixing whitespace issue

* Fixing whitespaces

* Fixing whitespaces

* Updating JavaDoc

Delegate enabled handling to Composite root. (#11832)

Otherwise the changed state isn't communicated properly to the
client-side in the initial round trip, as the client-side uses the child
connector's state directly.

Fixes #11831

Add support for excess slashes within static file request path. (#11827)

Fix issue where Chrome refused to select text in Table (#11709) (#11818)

Manual cherry-pick from Vaadin 7.

Updated local screenshot testing configurations. (#11814)

* A separate screenshot module hasn't been in use for a while but local
testing configurations and instructions weren't updated to the new model
at the time. Could use some further cleanup but this covers the basics.
* Some tweaks to ScreenshotBrowser to get it functional again, although
the actions for automatically replacing or adding alternative
screenshots still require work and have thus been disabled.

Don't attempt to send expand/collapse events for disabled TreeGrid. (#11823)

The attempt gets blocked later down the line in any case and never
reaches the server, but sending it messes up TreeGrid's internal state.
It gets stuck waiting for the resolution of the blocked call that can
then only be resolved by server-side sending its own expand/collapse
request programmatically. Until that happens no further expand/collapse
attempts will get sent to server even after the TreeGrid has been
enabled again.

Tree is also affected as it is built upon TreeGrid.

Fixes #11822

Decode path in getStaticFilePath (#11812)

* Decode path in getStaticFilePath

Some containers do not decode path when using getPathInfo, in case path has not been decoded there is a risk for path traversal vulnerability.

Fixing issue with TwinColSelect not correctly retaining visible selection (#11799)

There is a bug in TwinColSelect loging, it retains selection by indexes not by values after being sorted when new item has been added. This is a fixed by changing updateListBox method to retain the selection as it is being called after selection is being done.

Fixes: #11287

Eliminate un-intuitive NPE (#11802)

* Eliminate un-intuitive NPE

Fixes https://github.com/vaadin/framework/issues/10947

Cherry picks of Binder fixes in Flow (#11758)

* Cherry picks of Binder fixes in Flow

Addresses: https://github.com/vaadin/framework/issues/9000

Addresses:  https://github.com/vaadin/framework/issues/11109

These changes are adopted from https://github.com/vaadin/flow/pull/4138 and https://github.com/vaadin/flow/pull/6757

Reduce excess positioning calls for ComboBox popup. (#11808)

- If an open popup is reset to its default position on every update from
the server before getting adjusted again to the actual expected
position, on heavier applications some of those intermediate positions
might get rendered. If the ComboBox is positioned at the right edge and
the popup contents are longer than the input field (i.e. popup should
open to the left, not right) this might cause flickering.
- Setting the default position is only actually needed when the popup is
opened in order to give it a baseline, otherwise it's better to simply
adjust the position if needed.

Continues on #11718

Added missing constructor with scanNestedDefinitions option (#11801)

* Added missing constructor with scanNestedDefinitions option

Addresses https://github.com/vaadin/framework/issues/10949

Use APPLICATION_SCOPE for the session lock (#11792)

The Vaadin session itself is also stored in APPLICATION_SCOPE. The default
scope is PORTLET_SCOPE, so lock would be otherwise not be in sync with
the session.

To be able to do this, relevant methods in VaadinService are made protected so
that VaadinPortletService can override them.

Fixes #11611

Fixing issue with Push stopping working in some circumstances (#11791)

* Fixing issue with Push stopping working in some circumstances

If new request is attempted when resynchronization is ongoing, the Push will stop working. This patch fixes the issue by aborting handleJson if resynch is already ongoing.

This PR supercedes https://github.com/vaadin/framework/pull/11786

Fixes #11702, #7719

* Call onResynchronize() in MessageHandler

* Optimizing

Check actual Grid selection instead of relying on allSelected flag. (#11787)

The checkbox for selecting all rows only selects all the rows that have
not been filtered out. Changing the filtering does not change the
selection or the checkbox state so assuming that all rows are selected
simply because the checkbox has been checked cannot work.

Fixes #11479

Removed a duplicated word in a note. (#11785)

Added note to setTemplateContents JavaDoc (#11752)

Addresses https://github.com/vaadin/framework/issues/1262

Fix a timing issue in ComboBox filtering via paste using mouse. (#11780)

The filtering needs to be delayed, otherwise it's performed before the
new filter text is available and the old filter text is used instead.

Fixes #11779

Switch from an assert to logging and return in Grid.onBrowserEvent (#11778)

- If the Grid has frozen columns zooming can cause the regular column
cells to be ever so slightly out of sync with their corresponding rows.
This difference is not noticeable to naked eye but is big enough that
it's possible to hover over the row instead of the cell, which causes an
assertion error and a big ugly error popup. Switching to logging retains
the information delivered by the assertion error for developer purposes
but makes the end user experience smoother.
- Can be tested manually with GridColumnFrozenColumn test UI.

Fixes #11198

Added 1px tolerance to ScrollbarBundle's internal sanity check. (#11777)

* Added 1px tolerance to ScrollbarBundle's internal sanity check.

Requiring exact match can cause this check to fail when the browser is
zoomed since rounding is involved. This can in turn block some features
like opening of Grid Editor from working until some more scrolling
happens and the minute inconsistency is fixed.

Can be tested manually using GridEditorUI, depending on the environment
different amounts of zooming may be required for the problem to
manifest.

Fixes #11672

Improvements to ScrollDestination sanity checks (#11772)

- The new top row logical index should always be within the logical
range and high enough up to avoid leaving a gap if possible.
- Added regression testing for using the different scroll destination
types for scrolling to the top and to the bottom by index.

Fixes #11732

Added missing NOTIFICATION_DARK (#11770)

And corrected couple of typos

Addresses https://github.com/vaadin/framework/issues/11492

Fix the end limit of the allowed scrollTo rows. (#11771)

- Row index counts up from zero, data provider size counts up from one,
as one would expect. If the two match we are already past the available
range.

Made JavaDoc of updateSelection more precise (#11768)

Addresses https://github.com/vaadin/framework/issues/11520

Tweak a test to open Grid details row by clicking cell instead of row. (#11764)

Update chrome version to 78 (#11765)

When ComboBox popup opens to the left accommodate margin/border/padding. (#11755)

Fixes #11718

Make sure expanding/collapsing updates expected cache size. (#11753)

* Make sure expanding/collapsing updates expected cache size.

Fixes #11749

* Merge branch 'master' into issue11749

Make cancellation of uploads work regardless of Push configuration (#11743)

- Checking the push configuration outside of session lock threw
  an AssertionError, so the push configuration is not checked anymore.

- The original problem with cancelling Upload was due to a subtle
  ordering issue that depended on the Push configuration.
  In the case of PushMode.AUTOMATIC, a new StreamVariable was
  added by the `Upload` component _before_ the `FileUploadHandler`
  got a chance to remove the old `StreamVariable`. As a result, the
  `FileUploadHandler` actually removed the fresh `StreamVariable`,
  breaking future uploads.

Fixes #11682

Add note to setRenderer() JavaDoc about presentation provider (#11751)

Addresses https://github.com/vaadin/framework/issues/10277

Upload should preserve MIMEType after first selection (#11745)

* Upload should preserve MIMEType after first selection

Fix #11698

Automated test is not run locally, need to continue with it

* Remove unused imports

* Add sleep time in testing purposes

* Getting a new instance of an input, after file is downloaded

Exclude IE from tests. Throws https://stackoverflow.com/questions/23883071/unhandled-alert-exception-modal-dialog-present-selenium , but checked manually and it works

Make sure Firefox for iOS isn't detected as too old to function. (#11739)

Change-Id: Iea13b958356b0246586bcb7e9969bc26db43bb96

Catch and handle IllegalStateException (#11733)

* Catch and handle IllegalStateException

Fixes https://github.com/vaadin/framework/issues/11730

* Renaming variables

Added note in JavaDoc of removeColumn (#11725)

* Added note in JavaDoc of removeColumn

https://github.com/vaadin/framework/issues/11722

* Merge branch 'master' into fix11722

* Merge branch 'master' into fix11722

Update context menu addon version (#11735)

Update Chrome version to 77 (#11727)

* Update Chrome version to 77

* update screenshots

* update screenshots

Update release note to mention Java 11 support (#11697)

* Update release note to mention Java 11 support

* Update explanation about client side code

Removal of data communicator needs to happen before call to super.remove() (#11710)

Removal of data communicator needs to happen before call to super.remove() since super.remove() sets parent to null causing the NPE.

Fixes https://github.com/vaadin/framework/issues/11617

Correct usage of String.format(..) (#11704)

Fixes https://github.com/vaadin/framework/issues/11701

Remove double negation (#11713)

"doesn't not" should be either "doesn't" or "does not".

Fix scrollTo for destination START and END and add regression testing. (#11707)

- Initial implementation erroneously assumed that
ScrollDestination.START would only be used for scrolling up and
ScrollDestination.END for scrolling down. That's obviously not what they
are for, otherwise everyone would be using ScrollDestination.ANY.
- Moved actual scrolling to within the helper method that originally
only calculated the new scroll position. Parent method became too long
otherwise.

Fixes #11706

Updated row and spacer handling for Escalator (#11438)

Updated row and spacer handling for Escalator.

Main changes:
- Spacers are only maintained and checked for rows that have DOM
representation, and not at all if there is no details generator. This
gives notable performance improvements to some particularly large Grids
- Escalator no longer tries to trim away any rows that don't fit within
the viewport just because a details row gets opened in Grid. This leads
to some increase in simultaneous DOM elements, but simplifies the logic
considerably. For example opening or closing details rows doesn't
require checking the overall content validity beyond the details row
itself anymore, but some repositioning at most. There are also no longer
any orphaned spacers without corresponding DOM rows.
- Spacers are better integrated into the overall position calculations.
- Some public methods that are no longer used by Escalator or have
changed functionality or order of operations. Any extending classes that
tap into row, spacer, or scroll position handling are likely to need
reworking after this update.
- Auto-detecting row height is delayed until Escalator is both attached
and displayed.

Read-only styles shouldn't override borderless background styles. (#11692)

* Read-only styles shouldn't override borderless background styles.

- Fixed for ComboBox and DateField styles, other tested components
already do it right.

Fixes #11671

* Merge branch 'master' into issue11671

Add explicit handling of ctrl-a (#11693)

* Add explicit handling of ctrl-a

Fixes https://github.com/vaadin/framework/issues/11678

* Adds test case and formatting the code

Update Chrome version, snapshots and fix tests (#11684)

Add a missing letter (#11676)

Fixes https://github.com/vaadin/framework/issues/11409

Update the since tag (#11675)

* Update the since tag

Update Spring addon to 3.2.1 (#11670)

Update screenshot (#11669)

Apply offset to column index depending on presense of selection column (#11667)

* Apply offset to column index depending on presense of selection column

After testing this manually I noticed that offset correction needs to be negative

* Add the testing UI

Ensure the upload button has been disabled when set Enabled calling from succeedL… (#11655)

* Ensure the upload button has been disabled when calling from succeedListener with Push enabled

* Add Tests to verify

Prevent excessive getLogger() calls from markAsDirty() (#11664)

Fixes #10339

Update to 8.10-SNAPSHOT (#11657)

Use https for Maven repositories (#11668)

* Use https for Maven repositories

Ensure that VComboBox.selectedOptionKey gets initial selection. (#11665)

Fixes #10741

Improvements to detail row index handling. (#11345)

- Escalator should notify when an existing details row is moved to a new
index.
- Grid and DetailsManagerConnector should update their internal indexing
when details manager index changes in Escalator.