Closed Bug 751403 Opened 12 years ago Closed 12 years ago

Telephony tests: JavascriptException: TypeError: navigator is undefined

Categories

(Remote Protocol :: Marionette, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla15

People

(Reporter: jgriffin, Assigned: jgriffin)

Details

Attachments

(1 file)

I got the webapi tests running again, see http://brasstacks.mozilla.com/autolog/?tree=b2g&source=autolog.  Unfortunately, something broke the telephony tests between now and the last time they ran, 24 days ago.
The fix to all these tests is to replace navigator with window.navigator.  I don't know why this necessary now...I tried looking through commits but I couldn't identify any obvious candidates that would have caused this.

We could add the navigator property directly to the sandbox instead; I don't really have an opinion on which is better, do you?
Attachment #620508 - Flags: review?(mdas)
Comment on attachment 620508 [details] [diff] [review]
telephony tests patch

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

Looks good!

I would prefer not to include navigator in the sandbox if we don't have to. It makes the test unclear, and it makes the Marionette server responsible for things it shouldn't care about.
Attachment #620508 - Flags: review?(mdas) → review+
http://hg.mozilla.org/mozilla-central/rev/a0356446a56a
Assignee: nobody → jgriffin
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
(In reply to Jonathan Griffin (:jgriffin) from comment #1)
> The fix to all these tests is to replace navigator with window.navigator.  I
> don't know why this necessary now...I tried looking through commits but I
> couldn't identify any obvious candidates that would have caused this.

This seems like something worth investigating, lest it end up being caused by a more serious bug that causes more problems later. It shouldn't be necessary to prefix navigator with "window.".
(In reply to :Gavin Sharp (use gavin@gavinsharp.com for email) from comment #4)
> 
> This seems like something worth investigating, lest it end up being caused
> by a more serious bug that causes more problems later. It shouldn't be
> necessary to prefix navigator with "window.".

Gavin's right; this is a regression in gecko and not something to do with Marionette.  I'll file a separate bug.
Filed as bug 751699
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: