(7_3_X) [TIMOB-26325] Android: Ti.UI.WebView not firing events in 7.x #10273
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
JIRA: https://jira.appcelerator.org/browse/TIMOB-26325
Description:
This reverts the boolean flag hack to prevent multiple
load
/beforeLoad
events getting fired when changing theuserAgent
after construction.When running through
processProperties
we set the user agent string before loading the html/url/data content to avoid immediate reload.I think so long as you provide the user agent in the construction properties this should only ever fire the
load
/beforeLoad
events once. If you construct a WebView with an Url and then change the user agent after (mid-load), it's entirely possible you'll get multipleload
/beforeLoad
events fired based on the timing of the code.I included a quick blurb in the Ti.UI.WebView.userAgent apidocs about this behavior.