1037781 - Multiple CSS warnings shown when launching Camera app by swiping to the left at lockscreen

Status: RESOLVED INVALID

(Core :: CSS Parsing and Computation, defect)

Description

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

+++ This bug was initially created as a clone of Bug #1037780 +++

Turn on Lockscreen via Settings, turn off display, turn it on again. Instead of swiping right to unlock, swipe left to go into the Camera app instead. On or around the time the permissions dialog comes up - I chose not to share but also not remember this setting, the following JS warnings come up:

E/GeckoConsole(14426): [JavaScript Warning: "Unknown property 'speak'.  Declaration dropped." {file: "app://camera.gaiamobile.org/style/main.css" line: 1187 column: 6 source: "	speak:none;"}]
E/GeckoConsole(14426): [JavaScript Warning: "Unknown property '-moz-osx-font-smoothing'.  Declaration dropped." {file: "app://camera.gaiamobile.org/style/main.css" line: 1192 column: 24 source: "	-moz-osx-font-smoothing:grayscale;"}]
E/GeckoConsole(14426): [JavaScript Warning: "Unknown property 'will-change'.  Declaration dropped." {file: "app://camera.gaiamobile.org/index.html" line: 0 column: 0 source: "transform"}]

Setting needinfo? from Wilson, a Camera peer.

Tested on a Flame:
$ ./check_versions.sh
Gaia      c47094a26c87ba71a3da4bae54febd0da21f3393
Gecko     https://hg.mozilla.org/mozilla-central/rev/1b1296d00330
BuildID   20140711040202
Version   33.0a1
ro.build.version.incremental=109
ro.build.date=Mon Jun 16 16:51:29 CST 2014
B1TC00011220

Comment 1

[Wilson Page [:wilsonpage]]

These aren't JS warning but CSS. When Gecko encounters CSS properties that it doesn't understand it will log a warning. These aren't serious.

Gecko should comfortably be able to read CSS written for other engines (eg. WebKit) and not choke. This is the nature of the web. One code-base able to run on many different engines.

I am confused by the `will-change` warning though, AFAIK this property should be recognised and working.

NI Chris to clarify status of `will-change`...

Comment 2

[Chris Lord [:cwiiis]]

The will-change warning is odd - I think I've seen it before, even when it has actually taken effect, but that was a while ago and I don't know if I'm just misremembering.

BenWa should know more about will-change, shifting the n? to him.

Comment 3

[Benoit Girard (:BenWa)]

Will-change right now is special case to only be supported in privileged gaia apps. Maybe we incorrectly emit the warning then.

Comment 4

[Mike Habicher [:mikeh] (high bugzilla latency)]

Set component to CSS per comment 3.

Comment 5

[Boris Zbarsky [:bzbarsky]]

This has nothing to do with the cssom.

The warning people are seeing is what you'd get when you use a property whose name we don't recognize (though we don't warn about vendor-prefixed properties prefixed with something other than "-moz", to address comment 1.

The "speak" property is not supported at all.

The "-moz-osx-font-smoothing" property is only supported if the "layout.css.osx-font-smoothing.enabled" property is set, which is only on Mac.  I have no idea why gaia is using this at all.

The "will-change" property is enabled if the "layout.css.will-change.enabled" preference is set, unless you're either chrome or a certified app.

Is the camera app a certified app?

Comment 6

[Mike Habicher [:mikeh] (high bugzilla latency)]

Sorry about the wrong component. Yes, the Camera app on b2g is certified.

Comment 7

[Boris Zbarsky [:bzbarsky]]

That's quite odd.  Can someone reproduce this in an environment in which they can attach a debugger and see what's going on?

Comment 8

[Gregory Pappas [:gregp]]

Closing old B2G bugs