Closed Bug 1235740 Opened 6 years ago Closed 6 years ago

New startup terminal-spew: "Failed to load Oculus VR library!"

Categories

(Core :: Graphics, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox46 --- fixed

People

(Reporter: dholbert, Assigned: kip)

Details

Attachments

(1 file)

With my debug mozilla-inbound build (not sure about opt builds), I'm noticing the following output among the first few lines of terminal output, every time I start Firefox:
> Failed to load Oculus VR library!

(I'm on Ubuntu 15.10, and I've never setup any VR-specific stuff. Presumably there's some library we're looking for & failing to find on my system.)

I'm guessing this only just started appearing because we just recently preffed VR on by default, and we only look for these libs if VR is preffed on, it sounds like from bug 1113242.

I could see this message being helpful in the past when diagnosing issues with power-users who've opted in to trying VR and are likely to have the occulus toolkit installed.  But now that VR is preffed on by default, this is much more likely to just be background-noise / terminal-spam.

So: can we get rid of this error-message, or make it opt-in/better-targeted in some way?
Looks like this message comes from one of these printf_stderr spots:
 https://hg.mozilla.org/mozilla-central/annotate/9ddf0da90fb3/gfx/vr/gfxVROculus.cpp#l154
 https://hg.mozilla.org/mozilla-central/annotate/9ddf0da90fb3/gfx/vr/gfxVROculus050.cpp#l158

ni=vlad for how best to resolve this, since he's on 'hg blame' for this error-message & hence is in a position to evaluate its value.
Flags: needinfo?(vladimir)
(Actually, with e10s enabled [the default], this message gets spammed *twice* in my first few lines of terminal-output -- first from the chrome process, and again a bit later from the content process when it starts up [I assume].)
I can take this and remove the excess spew.  I believe this debug output is not needed any more.  WebVR has just been enabled in the latest nightly.
Assignee: nobody → kgilbert
Flags: needinfo?(vladimir)
Comment on attachment 8703011 [details]
MozReview Request: Bug 1235740 - Remove warning for missing Oculus VR Library from terminal output, as it is spammy and not necessary

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/29251/diff/1-2/
Attachment #8703011 - Flags: review?(dholbert)
Comment on attachment 8703011 [details]
MozReview Request: Bug 1235740 - Remove warning for missing Oculus VR Library from terminal output, as it is spammy and not necessary

https://reviewboard.mozilla.org/r/29251/#review26047

Looks good!

One nit: the commit message is a bit generic ("Remove debug output spam from terminal output").

Maybe it should say something like "Remove warning for missing Oculus VR Library", or something else that mentions Oculus and/or VR.

r=me regardless
Attachment #8703011 - Flags: review?(dholbert) → review+
Comment on attachment 8703011 [details]
MozReview Request: Bug 1235740 - Remove warning for missing Oculus VR Library from terminal output, as it is spammy and not necessary

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/29251/diff/1-2/
Attachment #8703011 - Attachment description: MozReview Request: Bug 1235740 - Remove debug output spam from terminal output → MozReview Request: Bug 1235740 - Remove warning for missing Oculus VR Library from terminal output, as it is spammy and not necessary
https://hg.mozilla.org/mozilla-central/rev/100f409cb8fa
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.