summaryrefslogtreecommitdiff
path: root/addons/web_editor/static/src
diff options
context:
space:
mode:
authorAlexandre Kühn <aku@odoo.com>2019-05-23 11:00:53 +0000
committerAlexandre Kühn <aku@odoo.com>2019-07-16 13:13:22 +0000
commit4bde721f925bfe3d1e867565490d3830da150dcc (patch)
tree33e646049c5c6b6cb201e87ad196d132c319c407 /addons/web_editor/static/src
parentcf8e74a1a7d3404f57798573e8b15ce4dc5b9be4 (diff)
[FIX] mail: notify other tabs on document thread message post
Before this commit, most new messages added to the mail manager were handled as if recently posted from a document thread. As a result, it was frequently setting a new item in the local storage to notify other tabs from these new messages. The intent of this behaviour is to notify other tabs of newly posted messages in a document thread, without using a longpolling notification. That means this logic should only apply on messages that have been recently posted on a document thread. Sometimes, this issue was breaking the local storage on Firefox. It showed the error message "Quota Exceeded Error" whenever `setItem` was used, even when items were removed beforehand. The capacity of the storage was only a few Kbs on this domain, far from reaching the maximum capacity. We think that this error comes from the concurrent `setItem` on multiple tabs, but this is very hard to reproduce. This commit fixes the issue by limiting the `setItem` to the tab that really posted the message. Tests have been adapted in order to pass only with this fix.
Diffstat (limited to 'addons/web_editor/static/src')
0 files changed, 0 insertions, 0 deletions