605351 - fix and re-enable browser_webconsole_bug_593003_iframe_wrong_hud.js

Status: RESOLVED FIXED

(DevTools :: General, defect, P3)

Description

[Rob Campbell [:rc] (:robcee)]

This test was disabled around the same time as the compartments landing, but I think was shut off separately from the original tests. We should get it fixed up and operational again.

Followup from bug 604536.

Comment 1

[Mihai Sucan [:msucan]]

Attachment: proposed test fix

Proposed test fix. All tests pass in debug and opt builds (Linux).

Comment 2

[David Dahl :ddahl]

Comment on attachment 485001 [details] [diff] [review]
proposed test fix

Is this patch missing the actual test bits where we verify things? It looks like it just opens tabs and closes them.

Comment 3

[Mihai Sucan [:msucan]]

(In reply to comment #2)
> Comment on attachment 485001 [details] [diff] [review]
> proposed test fix
> 
> Is this patch missing the actual test bits where we verify things? It looks
> like it just opens tabs and closes them.

This patch fixes the test. Indeed, in the patch the actual tests are not visible, however please apply the patch and see the resulting mochitest file. It opens two tabs and it tests if the iframe network request is recorded in the correct tab.

Comment 4

[Kevin Dangoor]

mass change: filter on PRIORITYSETTING

Comment 5

[Mihai Sucan [:msucan]]

This bug is already fixed, but I don't know by what other bug.

Comment 6

[Kevin Dangoor]

It's unfortunate that we don't know when this was fixed, but there's no point leaving this bug open if we know that it is fixed.