Closed Bug 961215 Opened 12 years ago Closed 11 years ago

Intermittent browser_tabview_bug625269.js | Much smaller: The group should have been resized - Got false, expected true

Categories

(Firefox Graveyard :: Panorama, defect)

Product:
Firefox Graveyard
Component:
Panorama
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox38 wontfix, firefox38.0.5 fixed, firefox39 fixed, firefox40 fixed, firefox-esr31 unaffected, firefox-esr38 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 40
Tracking Flags:
Tracking Status
firefox38 --- wontfix
firefox38.0.5 --- fixed
firefox39 --- fixed
firefox40 --- fixed
firefox-esr31 --- unaffected
firefox-esr38 --- fixed

People

(Reporter: RyanVM, Assigned: ttaubert)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

0001-Bug-961215-Fix-intermittent-browser_tabview_bug62526.patch
1.39 KB, patch
MattN
: review+
Details | Diff | Splinter Review
https://tbpl.mozilla.org/php/getParsedLog.php?id=33186135&tree=B2g-Inbound Ubuntu VM 12.04 x64 b2g-inbound opt test mochitest-browser-chrome on 2014-01-17 11:23:11 PST for push cdafc366803c slave: tst-linux64-spot-426 11:43:10 INFO - TEST-START | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js 11:43:10 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | Only one tab exist 11:43:10 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | TabContextMenu context is the expected tab 11:43:10 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | The 'Close tab' menu item is disabled 11:43:10 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | The 'Move to New Window' menu item is disabled 11:43:10 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | Has one group only 11:43:10 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | 2 11:43:10 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | TabContextMenu context is the expected tab 11:43:10 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | The 'Close tab' menu item is enabled 11:43:10 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | The 'Move to New Window' menu item is enabled 11:43:10 INFO - TEST-INFO | MEMORY STAT vsize after test: 1313193984 11:43:10 INFO - TEST-INFO | MEMORY STAT residentFast after test: 358133760 11:43:10 INFO - TEST-INFO | MEMORY STAT heapAllocated after test: 143284920 11:43:10 INFO - INFO TEST-END | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625195.js | finished in 783ms 11:43:10 INFO - TEST-INFO | checking window state 11:43:10 INFO - TEST-START | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js 11:43:11 INFO - console.error: 11:43:11 INFO - [CustomizableUI] 11:43:11 INFO - Could not localize property 'test-removable-widget-default2.label'. 11:43:11 INFO - console.error: 11:43:11 INFO - [CustomizableUI] 11:43:11 INFO - Could not localize property 'test-removable-widget-default2.tooltiptext'. 11:43:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js | A little smaller: The group should not have been resized 11:43:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js | A little bigger: The group should not have been resized 11:43:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js | Much smaller: The group should have been resized 11:43:12 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js | Bigger after much smaller: The group should have been resized 11:43:12 WARNING - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js | Much smaller: The group should have been resized - Got false, expected true 11:43:12 INFO - Stack trace: 11:43:12 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 705 11:43:12 INFO - JS frame :: chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js :: onTabViewShown/checkResized/< :: line 23 11:43:12 INFO - JS frame :: chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js :: tryResize :: line 73 11:43:12 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: testScope/test_executeSoon/<.run :: line 734 11:43:12 INFO - native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0 11:43:12 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js | After clearing the cramp: The group should not have been resized 11:43:12 INFO - TEST-INFO | MEMORY STAT vsize after test: 1314373632 11:43:12 INFO - TEST-INFO | MEMORY STAT residentFast after test: 362057728 11:43:12 INFO - TEST-INFO | MEMORY STAT heapAllocated after test: 152081160 11:43:12 INFO - INFO TEST-END | chrome://mochitests/content/browser/browser/components/tabview/test/browser_tabview_bug625269.js | finished in 1320ms
Closing bugs where TBPLbot has previously commented, but have now not been modified for >3 months & do not contain the whiteboard strings for disabled/annotated tests or use the keyword leave-open. Filter on: mass-intermittent-bug-closure-2014-07
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WORKSFORME
Reopening as per recent comments.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Holy recent huge spike Batman.
Flags: needinfo?(ttaubert)
Turns out that window.resizeTo() can "fail" sometimes. The right size is applied but for some reason a native resize event overrides it with the old value. At the time the "resize" DOM event is dispatched the window is back to its old size and we basically have to call it again.
Assignee: nobody → ttaubert
Status: REOPENED → ASSIGNED
Flags: needinfo?(ttaubert)
Attachment #8602954 - Flags: review?(MattN+bmo)
Iteration: --- → 40.3 - 11 May
Points: --- → 3
Flags: qe-verify-
Flags: firefox-backlog+
OS: Linux → All
Hardware: x86_64 → All
Attachment #8602954 - Flags: review?(MattN+bmo) → review+
(In reply to Treeherder Robot from comment #149) > log: > https://treeherder.mozilla.org/logviewer.html#?repo=fx-team&job_id=2993286 > repository: fx-team > start_time: 2015-05-08T06:49:03 > who: tomcat[at]mozilla[dot]com > machine: t-w732-ix-144 > buildname: Windows 7 32-bit fx-team opt test mochitest-browser-chrome-1 > revision: 201c323dcf16 That was before my push, all good so far.
This latest explosion was from bug 1149555, now backed out.
https://hg.mozilla.org/mozilla-central/rev/aa1011c251a8
Status: ASSIGNED → RESOLVED
Closed: 12 years ago → 11 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: