Last Comment Bug 803126 - google.com/green has broken sniffing for the prefix to use with "transform", ends up trying to set "trueTransform"
: google.com/green has broken sniffing for the prefix to use with "transform", ...
Status: RESOLVED INVALID
[webkitjs] [country-all]
:
Product: Tech Evangelism
Classification: Other
Component: Desktop (show other bugs)
: unspecified
: x86_64 Windows 7
: -- normal (vote)
: ---
Assigned To: Karl Dubost :karlcow
:
Mentors:
http://www.google.com/green/storyofse...
Depends on:
Blocks: 745523
  Show dependency treegraph
 
Reported: 2012-10-18 08:26 PDT by Alice0775 White
Modified: 2014-10-11 20:58 PDT (History)
9 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
-
-
-
-


Attachments

Description Alice0775 White 2012-10-18 08:26:03 PDT
Build Identifier:
http://hg.mozilla.org/releases/mozilla-release/rev/0507d387617c
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/20100101 Firefox/16.0 ID:20121010144125

This problem happens since Bug 745523 landing.
 
Steps to reproduce:
1. Open http://www.google.com/green/storyofsend/desktop/#/it-starts-with-send

Actual results:
Sound is played back as expected.
However, no graphics is not drawn.
Comment 1 Boris Zbarsky [:bz] 2012-10-18 08:44:39 PDT
Looks like the usual "Assume things are prefixed" thing.  For example:

            function a() {
                var a = b("transform"),
                    c = a + "Perspective" in document.documentElement.style || n;
                return a ? {
                    vendor: a,
                    Kb: c
                } : n
            }
            function b(a) {
                var c = document.createElement("div"),
                    b = ["Khtml", "Ms", "O", "Moz", "Webkit"],
                    d = b.length;
                if (a in c.style) return k;
                for (a = a.replace(/^[a-z]/, function (a) {
                    return a.toUpperCase()
                }); d--;) if (b[d] + a in c.style) return b[d];
                return n
            }

So b("transform") will return k.  k is the JS value true.  That will then test for "truePerspective" in document.documentElement.style, end up with false, and set "vendor" to true and "Kb" to n.  n is false.

Then this will go through and do things like set document.documentElement.className to "vendortrue" and worse yet get to this.wb which does:

        this.wb = function (b) {
            if (a.N) {
                var c = a.N.vendor + "Transform",
                    d = O.ta(b.Sa, b.Ta);
                b.a.style[c] = d
            } else {
                b.a.style.top = b.Ta + "px";
                b.a.style.left = b.Sa + "px"
            }
            b.$ = b.Sa;
            b.aa = b.Ta
        };

which will try to set "trueTransform" on b.a.style, which will of course do absolutely nothing.

Looks pretty similar to the script we saw in bug 790915, actually.
Comment 2 Lukas Blakk [:lsblakk] use ?needinfo 2012-10-18 12:23:08 PDT
Dbaron, if this is pretty similar to what we say in bug 790915 and I'm reading your comment https://bugzilla.mozilla.org/show_bug.cgi?id=790915#c16 correctly, then it seems that we actually don't have to track this and that this becomes a tech evangelism issue - is that correct or am I missing something?
Comment 3 Boris Zbarsky [:bz] 2012-10-18 12:23:55 PDT
I mailed the Google folks about this, by the way.
Comment 4 Lukas Blakk [:lsblakk] use ?needinfo 2012-10-19 15:21:26 PDT
This isn't a critical website regression so we'll untrack, people will have to do outreach/evangelism on this but we won't block a release on it.
Comment 5 Boris Zbarsky [:bz] 2012-10-19 17:35:23 PDT
So what I know so far is that both this site and the site in bug 790915 were apparently done by the same company: B-Reel.  http://www.b-reel.com/digital/projects/story-of-send/ for the Google end of this.

So what we should probably do is contact them and find out what other things they've done, since I bet everything they've done will break this way.  :(

I can't tell whom to contact from their contact page.  Lukas, do you think you can get hold of someone who knows how to talk to outfits like this and get them to do it?
Comment 6 Boris Zbarsky [:bz] 2012-11-04 21:17:28 PST
Johnny, do you know how we can get someone who can deal with this to look at it?
Comment 7 Karl Dubost :karlcow 2014-03-10 23:29:43 PDT
Confirmation that there is still no graphics there. Moving this to Desktop.
Comment 8 Hallvord R. M. Steen [:hallvors] 2014-10-11 02:21:25 PDT
Now renders like in Chrome for me - was it fixed?
Comment 9 Karl Dubost :karlcow 2014-10-11 20:58:41 PDT
With Firefox 32 on Desktop 
Going to http://www.google.com/green/storyofsend/desktop/#/it-starts-with-send
Redirect to http://www.google.com/green/#/it-starts-with-send

I don't hear any sounds, but all graphics seem to be displayed as planned.
Maybe the Web site changed since the initial design.

We don't have screenshots.
But if I go to 
http://web.archive.org/web/20130325090336/http://www.google.com/green/storyofsend/desktop/

We can note that the site was different. This section doesn't exist anymore. 
Let's close as INVALID.

Note You need to log in before you can comment on or make changes to this bug.