769266 - Add an option to log to Console when animations can't be asyncified.

Status: RESOLVED DUPLICATE of bug 1121896

(Core :: Graphics: Layers, defect)

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

Please make sure that these notifications are visible through the Web Console.

Comment 2

[David Zbarsky (:dzbarsky)]

That's the eventual plan, just need to make sure we don't spam the console too much.

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

Can we just add an INFO message for CSS that states:

"Some CSS animations could not be drawn asynchronous, potentially having performance implications: color, -moz-awesome-property, and -moz-awesome-property2."

And only do that when we parse the CSS (instead of printing this each time an animation begins).

Comment 4

[David Zbarsky (:dzbarsky)]

Attachment: Patch

Comment 5

[David Zbarsky (:dzbarsky)]

Attachment: PAtch

Comment 6

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Comment on attachment 647616 [details] [diff] [review]
PAtch

You want to use 

  nsCOMPtr<nsIConsoleService> consoleService =
    do_GetService(NS_CONSOLESERVICE_CONTRACTID);
  consoleService->LogStringMessage(msg);

Comment 7

[Joe Drew (not getting mail)]

Do we usually get these error messages localized?

Comment 8

[David Zbarsky (:dzbarsky)]

Attachment: Patch using console service

Comment 9

[Joe Drew (not getting mail)]

Er, I guess I should have asked that question of Axel.

Axel, do we usually get web developer-specific console notifications localized?

Comment 10

[Axel Hecht [:Pike]]

Yes, we would localize messages "like this". Please get some help on the actual message, though. I dare say that for people without physical access to your brain, that message isn't helpful. Too much jargon and acronyms. Something like jaws suggested in comment 3 is much more digestible, and quite useful to be localized. You may want to add a localization note on phrases that shouldn't be translated, like CSS keywords, if that makes sense. https://developer.mozilla.org/en/Localization_notes has tips on how to mark those up. Technically, I think we put the strings for code in layout into dom/locales/en-US.

Comment 11

[Jared Wein [:jaws] (please needinfo? me)]

Can you use an about:config pref for this? I'm not sure how PR_GetEnv works, so maybe it already uses about:config prefs.

Comment 12

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.