Closed Bug 881260 Opened 11 years ago Closed 11 years ago

Intermittent browser_bookmarks.js | Delete button is visible. | runTests: Task failed

Categories

(Firefox for Metro Graveyard :: Tests, defect, P3)

x86_64
Windows 8.1
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 24

People

(Reporter: jimm, Assigned: rsilveira)

References

Details

(Keywords: intermittent-failure, Whiteboard: feature=defect c=testing u=developer p=0)

Attachments

(1 file)

05:28:11 WARNING - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/metro/browser/metro/base/tests/mochitest/browser_bookmarks.js | Delete button is visible. 05:28:11 WARNING - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/metro/browser/metro/base/tests/mochitest/browser_bookmarks.js | runTests: Task failed - Error: Timed out waiting for condition to be true at testCondition@chrome://mochitests/content/metro/browser/metro/base/tests/mochitest/head.js:339 https://tbpl.mozilla.org/php/getParsedLog.php?id=23957922&tree=Mozilla-Central
Assignee: nobody → rsilveira
Status: NEW → ASSIGNED
Attached patch Patch v1Splinter Review
Added a waitForCondition, trying fix on: https://tbpl.mozilla.org/?tree=Try&rev=832208a9ff53
Comment on attachment 761727 [details] [diff] [review] Patch v1 I guess if something is really broken, we'll time out here instead? Or do you think we're just querying for button state too soon?
Attachment #761727 - Flags: review?(jmathies) → review+
I think we're just querying too soon.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24
(In reply to TinderboxPushlog Robot from comment #10)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
resolving thanks to a lack of reports since 6-14.
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Blocks: metro-testing, 865451, metrov1defect&change
No longer blocks: 880298
Priority: -- → P3
Whiteboard: feature=defect c=testing u=developer p=0
Blocks: 880298
No longer blocks: metro-testing
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: