Implement a way to queue sending feedback reports started within short intervals of each others |
|||||
Issue descriptionCurrently, if two feedback reports were requested to be sent successively within a very short time from each other, the first report might not be sent, and the second might be sent with the sys info of the first one. We need to implement a way to queue both reports correctly.
,
Mar 10 2016
,
Mar 17 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9c678f28bc8b50041b8aaa700714c75cd02b19f4 commit 9c678f28bc8b50041b8aaa700714c75cd02b19f4 Author: afakhry <afakhry@chromium.org> Date: Thu Mar 17 19:13:12 2016 Fix sending multiple feedback reports within short durations of each other Those reports needed to be queued nicely and each report should be sent with its correct respective system information not the one of the previous report. BUG= 593452 Test=manually Review URL: https://codereview.chromium.org/1794513002 Cr-Commit-Position: refs/heads/master@{#381768} [modify] https://crrev.com/9c678f28bc8b50041b8aaa700714c75cd02b19f4/chrome/browser/extensions/api/feedback_private/feedback_service.cc [modify] https://crrev.com/9c678f28bc8b50041b8aaa700714c75cd02b19f4/chrome/browser/extensions/api/feedback_private/feedback_service.h [modify] https://crrev.com/9c678f28bc8b50041b8aaa700714c75cd02b19f4/chrome/browser/resources/feedback/js/event_handler.js [modify] https://crrev.com/9c678f28bc8b50041b8aaa700714c75cd02b19f4/chrome/browser/resources/feedback/js/feedback.js
,
Mar 17 2016
,
Apr 11 2016
,
May 23 2016
Bulk verified |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by r...@chromium.org
, Mar 9 2016