success in PopupMenuImpl.cpp |
||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=5073893090131968 Fuzzer: inferno_layout_test_unmodified Job Type: linux_debug_content_shell_drt Platform Id: linux Crash Type: CHECK failure Crash Address: Crash State: success in PopupMenuImpl.cpp blink::PopupMenuImpl::setValueAndClosePopup blink::PagePopupController::setValueAndClosePopup Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_content_shell_drt&range=384595:384629 Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv96Ggj6ob1OalCJ50GUwXSqx5UHfxrrFxg6hH8RzLh_qKcrTgeTOnEPvfN63Z_wIWC8Q8HEQo31rnZmDnXvGfKc8siKHfyGAcaWAJej4up9oMXD1H-DVMRQ90aqUtjByAu2KZoFR8bWD7eMoUQ_UPgnE2C7fpQ?testcase_id=5073893090131968 Additional requirements: Requires Gestures Filer: nyerramilli See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Jul 20 2016
When Tab or Enter is pressed without selection, we call setValueAndClosePopup() with an empty string. It causes a DCHECK failure.
However this bug won't cause serious problems.
ListPicker.prototype._handleKeyDown = function(event) {
...
} else if (key === "Tab" || key === "Enter") {
window.pagePopupController.setValueAndClosePopup(0, this._selectElement.value);
,
Sep 21 2016
,
Sep 21 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/676be06451fec4d4f2f0bab075f3b627ec1d1597 commit 676be06451fec4d4f2f0bab075f3b627ec1d1597 Author: tkent <tkent@chromium.org> Date: Wed Sep 21 07:02:46 2016 Fix a DCHECK failure in PopupMenuImpl::setValueAndClosePopup(). |stringValue| may be empty. BUG= 629029 Review-Url: https://codereview.chromium.org/2350303005 Cr-Commit-Position: refs/heads/master@{#419991} [modify] https://crrev.com/676be06451fec4d4f2f0bab075f3b627ec1d1597/third_party/WebKit/LayoutTests/fast/forms/select-popup/popup-menu-key-operations.html [modify] https://crrev.com/676be06451fec4d4f2f0bab075f3b627ec1d1597/third_party/WebKit/LayoutTests/platform/win/fast/forms/select-popup/popup-menu-key-operations-expected.txt [modify] https://crrev.com/676be06451fec4d4f2f0bab075f3b627ec1d1597/third_party/WebKit/Source/web/PopupMenuImpl.cpp
,
Sep 22 2016
ClusterFuzz has detected this issue as fixed in range 419971:420014. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5073893090131968 Fuzzer: inferno_layout_test_unmodified Job Type: linux_debug_content_shell_drt Platform Id: linux Crash Type: CHECK failure Crash Address: Crash State: success in PopupMenuImpl.cpp blink::PopupMenuImpl::setValueAndClosePopup blink::PagePopupController::setValueAndClosePopup Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_content_shell_drt&range=384595:384629 Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_content_shell_drt&range=419971:420014 Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv96Ggj6ob1OalCJ50GUwXSqx5UHfxrrFxg6hH8RzLh_qKcrTgeTOnEPvfN63Z_wIWC8Q8HEQo31rnZmDnXvGfKc8siKHfyGAcaWAJej4up9oMXD1H-DVMRQ90aqUtjByAu2KZoFR8bWD7eMoUQ_UPgnE2C7fpQ?testcase_id=5073893090131968 Additional requirements: Requires Gestures See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information. If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
,
Sep 22 2016
ClusterFuzz testcase is verified as fixed, closing issue. If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
,
Oct 18 2016
,
Nov 22 2016
Removing EditIssue view restrictions from ClusterFuzz filed bugs. If you believe that this issue should still be restricted, please reapply the label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by nyerramilli@chromium.org
, Jul 18 2016Components: Tools>Test>FindIt>NoResult
Labels: findit-wrong Te-Logged M-52
Owner: tkent@chromium.org
Status: Assigned (was: Available)