Closed Bug 1120592 Opened 9 years ago Closed 9 years ago

Intermittent test_browserElement_inproc_CopyPaste.html | uncaught exception - TypeError: iframeInner is null at http://mochi.test:8888/tests/dom/browser-element/mochitest/browserElement_CopyPaste.js:71

Categories

(Core :: DOM: Content Processes, defect)

Product:
Core
Component:
DOM: Content Processes
Platform:
x86_64
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla40
Tracking Flags:
Tracking Status
firefox37 --- wontfix
firefox38 --- fixed
firefox38.0.5 --- fixed
firefox39 --- fixed
firefox40 --- fixed
firefox-esr31 --- unaffected
firefox-esr38 --- fixed
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: RyanVM, Assigned: mtseng)

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 3 obsolete files)

Create iframe directly instead of using setTimeout v4. (carry r+: kanru)
3.60 KB, patch
Details | Diff | Splinter Review 
04:56:28 INFO - 1368 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | event.detail.width is not zero (test: normal div)
04:56:28 INFO - 1369 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | event.detail.height is not zero (test: normal div)
04:56:28 INFO - 1370 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | event.detail.state selectall
04:56:28 INFO - 1371 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | Clipboard has the correct value
04:56:28 INFO - 1372 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | copy command works (test: normal div)
04:56:28 INFO - 1373 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | paste command works (test: normal div)
04:56:28 INFO - 1374 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | Clipboard has the correct value
04:56:28 INFO - 1375 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | cut function works (test: normal div)
04:56:28 INFO - 1376 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | cut command works (test: normal div)
04:56:28 INFO - 1377 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | got mozbrowserselectionstatechanged event. (test: normal div with designMode:on)
04:56:28 INFO - 1378 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | event.detail is not null. (test: normal div with designMode:on)
04:56:28 INFO - 1379 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | event.detail.width is not zero (test: normal div with designMode:on)
04:56:28 INFO - 1380 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | event.detail.height is not zero (test: normal div with designMode:on)
04:56:28 INFO - 1381 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | event.detail.state selectall
04:56:28 INFO - 1382 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | Clipboard has the correct value
04:56:28 INFO - 1383 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | copy command works (test: normal div with designMode:on)
04:56:28 INFO - 1384 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | paste command works (test: normal div with designMode:on)
04:56:28 INFO - 1385 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | Clipboard has the correct value
04:56:28 INFO - 1386 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | cut function works (test: normal div with designMode:on)
04:56:28 INFO - 1387 INFO TEST-PASS | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | cut command works (test: normal div with designMode:on)
04:56:28 INFO - 1388 INFO TEST-UNEXPECTED-FAIL | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | uncaught exception - TypeError: iframeInner is null at http://mochi.test:8888/tests/dom/browser-element/mochitest/browserElement_CopyPaste.js:71
04:56:28 INFO - JavaScript error: http://mochi.test:8888/tests/dom/browser-element/mochitest/browserElement_CopyPaste.js, line 71: TypeError: iframeInner is null
04:56:28 INFO - ############################### browserElementPanning.js loaded
04:56:28 INFO - ######################## BrowserElementChildPreload.js loaded
04:56:28 INFO - 1389 INFO MEMORY STAT vsize after test: 4235718656
04:56:28 INFO - 1390 INFO MEMORY STAT residentFast after test: 637632512
04:56:28 INFO - 1391 INFO MEMORY STAT heapAllocated after test: 104582184
04:56:28 INFO - 1392 INFO TEST-OK | dom/browser-element/mochitest/test_browserElement_inproc_CopyPaste.html | took 1882ms
Morris, can you please take a look at this frequent intermittent failure?
status-b2g-v2.2: --- → affected
status-b2g-master: --- → affected
status-firefox37: --- → wontfix
status-firefox38: --- → affected
status-firefox39: --- → affected
status-firefox40: --- → affected
status-firefox-esr31: --- → unaffected
status-firefox-esr38: --- → affected
Flags: needinfo?(mtseng)
If we create iframe in callback of setTimeout, we might missing mozbrowserloadend. So create iframe directly.
Attachment #8596377 - Flags: review?(kchen)
Fix typo in commit message.
Attachment #8596377 - Attachment is obsolete: true
Attachment #8596377 - Flags: review?(kchen)
Attachment #8596378 - Flags: review?(kchen)
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #107)
> Morris, can you please take a look at this frequent intermittent failure?

Sure.
Assignee: nobody → mtseng
Status: NEW → ASSIGNED
Flags: needinfo?(mtseng)
Comment on attachment 8596378 [details] [diff] [review]
Create iframe directly instead of using setTimeout v2.

Review of attachment 8596378 [details] [diff] [review]:
-----------------------------------------------------------------

How are you sure the iframeInner "mozbrowserloadend" event is not dispatched before iframeOuter's?
Addressed kanru's comment.
Attachment #8596378 - Attachment is obsolete: true
Attachment #8596378 - Flags: review?(kchen)
Attachment #8596494 - Flags: review?(kchen)
Comment on attachment 8596494 [details] [diff] [review]
Create iframe directly instead of using setTimeout v3.

Review of attachment 8596494 [details] [diff] [review]:
-----------------------------------------------------------------

file_NestedFramesOuter_CopyPaste.html could be replaced by file_empty.html
You could use iframe.src = browserElementTestHelpers.emptyPage1 in the code.
Attachment #8596494 - Flags: review?(kchen) → review+
Use file_empty.html and remove file_NestedFramesOuterCopyPaste.html.
Attachment #8596494 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/e9ef9a74e0dc
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox40: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: