679025 - finance.google.com presents null error on initial load

Status: RESOLVED FIXED

(Web Compatibility :: Site Reports, defect, P3)

Description

[[e-mail] [:onecyrenus]]

finance.google.com charts don't seem to load on firefox beta on my android device. It used to work around 2 betas ago.

Comment 1

[[e-mail] [:onecyrenus]]

A repro case would be to go to http://finance.google.com and enter in a stock symbol.  No graph / chart will display. This works fine on desktop.

Comment 2

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Not sure exactly what's going on, it may be a finance.google.com issue?  I can get to the site via doing a google search and the stock I want at www.google.com but not at finance.google.com for some reason.

Comment 3

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Looking at view-source on the phone and for the desktop and comparing, I found that the view-source for the mobile is missing a good chunk of code.  One of which is the graph rendering script call ( <script>google.finance.renderMarketData();</script> ) which explains why the graph is missing.

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

with the latest nightly you just go directly to the website and it won't show the graph:
1. http://www.google.com/finance?q=NASDAQ:GOOG

Note:
Mozilla/5.0 (Android; Linux armv7I; rv9.0a1) Gecko/20110826 Firefox/9.0a1 Fennec/9.0a1
Device: Thunderbolt
OS: Android 2.2

Comment 5

[Jason Smith [:jsmith]]

Note - This is still not working as of 4/5/2012 for Fennec Native Aurora and Nightly. Looks like null is being returned on the user interface for Fennec Native. Stock browser still renders it correctly.

Comment 6

[Jason Smith [:jsmith]]

Attachment: Fennec Native Google Finance

Comment 7

[Jason Smith [:jsmith]]

Attachment: Stock Browser Google Finance

Comment 8

[Jason Smith [:jsmith]]

Attachment: Google Finance Opera

Comment 9

[Jason Smith [:jsmith]]

Actually dug into this a bit more - If you search for a chart in google's search engine, charts are actually loading. There does appear to be UA sniffing evident though going on with this site in general.

Comment 10

[Aaron Train [:aaronmt]]

Although the front-page seems to be broken on their end; one can still search I've noticed in the bottom drop-down and input fields: e.g, http://www.google.com/m/search/?sa=2&q=td&site=finance.

Ideally we want their mobile optimized touch version; and in the short-term they should fix the front-page of their finance app in their UA sniffing.

Comment 11

[Jason Smith [:jsmith]]

Since this is a not a CSS-prefixes issue the layout team can immediately control, this no longer affects bug 739832.

Comment 12

[Jason Smith [:jsmith]]

Nominating for kilimanjaro, as this relates to a top google app.

Comment 13

[Aaron Train [:aaronmt]]

As if June 14th, 2012 - they are now serving a sub-optimal basic HTML version. As I understand it, we're really pushing for their touch-webkit based version.

With this version the issue is reintroduced.

Comment 16

[Aaron Train [:aaronmt]]

Do we have an update here?

Comment 17

[Hallvord R. M. Steen [:hallvors]]

Karl, is this on your radar?

Comment 18

[Karl Dubost💡 :karlcow]

hehe it was not. :) I will check that. Thanks for the heads up Hallvord.

Comment 19

[Karl Dubost💡 :karlcow]

http://www.google.com/finance?cid=22144

On Android   Firefox      graph not displayed, just an empty frame, almost no content.
On ZTE       Firefox OS   graph not displayed, just an empty frame, almost no content.
On Android   Opera Mobile all content displayed. everything is fine.
On iPodTouch Safari       everything is fine (desktop content)

Working on Presto and Webkit but not on Gecko it seems.


# iOS and Opera Mobile on Android receives similar content

http GET http://www.google.com/finance?cid=22144 User-Agent:'Mozilla/5.0 (iPhone; CPU iPhone OS 5_0 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9A334 Safari/7534.48.3' | grep -i mobile

http GET http://www.google.com/finance?cid=22144 User-Agent:'Opera/12.02 (Android 4.1; Linux; Opera Mobi/ADR-1111101157; U; en-US) Presto/2.9.201 Version/12.02' | grep -i mobile


# Firefox OS and Firefox Android receives different not working content

http GET http://www.google.com/finance?cid=22144 User-Agent:'Mozilla/5.0 (Mobile; rv:18.0) Gecko/18.0 Firefox/18.0' | grep -i mobile

http GET http://www.google.com/finance?cid=22144 User-Agent:'Mozilla/5.0 (Android; Mobile; rv:18.0) Gecko/18.0 Firefox/18.0' | grep -i mobile

Comment 20

[Karl Dubost💡 :karlcow]

When entering http://finance.google.com/

On ZTE       Firefox OS   redirected to http://www.google.com/finance
On Android   Firefox      redirected to https://www.google.com/m/finance#markets
On Android   Opera Mobile redirected to https://www.google.com/m/finance#markets
On iPodTouch Safari       redirected to https://www.google.com/m/finance#markets

On ZTE Firefox back to issue on Comment #19
On Android/Firefox  Infinite wheel displayed under the top menu.
On others a stock market chart is displayed and some news.

When manually entering, on ZTE Firefox, https://www.google.com/m/finance#markets the device receives a page saying "No results found for null" :)

Comment 21

[Karl Dubost💡 :karlcow]

Ah and last but not least :)

on Opera Mini Android
http://finance.google.com/ redirects to http://www.google.com/finance with full content and graphs

and with http://www.google.com/finance?cid=22144
It's displaying everything perfectly

Comment 22

[Karl Dubost💡 :karlcow]

Contacted our usual contact at Google.

Comment 23

[Karl Dubost💡 :karlcow]

Firefox OS 1.2  still receive a lowfi version
Firefox Android still receives nothing with spinning wheel

Not fixed as of today.

Comment 24

[chris hofmann]

looks like it works fine for me in fennec (using beta 29).

not so good when trying to view portfolios on firefoxOS (using flame 28.0 2014-04-13)

should I file another bug for that or should that be tracked here?

Comment 25

[Hallvord R. M. Steen [:hallvors]]

I'd prefer a new bug, thanks.

When loading finance.google.com in Firefox for Android now, I see a message about Google Finance beta being available in certain other countries. So this may be fixed in their beta site, but it seems the site I get still has problems and missing content.

Comment 26

[chris hofmann]

filed Bug 1002690 for firefox OS

Comment 27

[Karl Dubost💡 :karlcow]

See https://bugzilla.mozilla.org/show_bug.cgi?id=754750#c14
This might be invalid now.

Comment 28

[Karl Dubost💡 :karlcow]

closing as INVALID according to 
https://bugzilla.mozilla.org/show_bug.cgi?id=754750#c15

Comment 29

[Luke Wagner [:luke]]

FWIW, I just tried on FF Nightly and Release on a Nexus 5 and the main page has a blank hole where there should be the market graphs.

Comment 30

[Karl Dubost💡 :karlcow]

Luke,
yes it has always been the case. It's what the bug was about. Maybe I close too fast if we consider that we should try at least to get the desktop version working on mobile. 

fwiw, I can ask Google to send us exactly the same content they send to Chrome, because there is no mobile version and this version is working on Firefox.

Comment 31

[Karl Dubost💡 :karlcow]

Recontacted Google today.

Comment 32

[Karl Dubost💡 :karlcow]

Attachment: google-finance-deviceS.jpg

Adding a comparison screenshot to show that it doesn't make sense.

Comment 33

[Karl Dubost💡 :karlcow]

Recontacted Google today with the screenshot and the network error happening.


16:22:31.009 GET http://www.google.com/finance [HTTP/1.1 200 OK 453ms]
16:22:31.069 GET http://www.google.com/finance/qs/channel [HTTP/1.1 200 OK 3069ms]
16:22:31.201 GET http://www.google.com/finance/f/finance_us-3578168175.css [HTTP/1.1 304 Not Modified 43ms]
16:22:31.201 GET http://www.google.com/finance/_/ss/a/ver=1q92easqloa2i/am=!KRcP96gTuVV0rbxY/bf=woE/r=O [HTTP/1.1 304 Not Modified 40ms]
16:22:33.981 GET http://www.gstatic.com/og/_/js/k=og.og.en_US.yyhlCiEji3A.O/rt=j/m=ld,sy31,sy32,sy34,sy35,sy36,sy37,sy38,sy39,sy44,sy40,sy41,sy42,sy43,d,sy57,gl,is,sy48,sy46,sy51,sy52,id,nb,nw,sb,sd,st,sy47,awd,sy50,sy54,p,vd,lod,eld,ip,dp,cpd/rs=AItRSTNAOxHgCAhCBIDi5y5--Tv9dLd43A [HTTP/1.1 304 Not Modified 45ms]
16:22:33.981 GET http://www.google-analytics.com/analytics.js [HTTP/1.1 304 Not Modified 17ms]
16:22:33.982 GET http://ssl.gstatic.com/gb/images/v1_376447c3.png [HTTP/1.1 304 Not Modified 69ms]
16:22:33.982 GET http://www.google.com/finance/f/sfe-opt-3505992447.js [HTTP/1.1 304 Not Modified 28ms]
16:22:33.898 99 d=0:1:46
16:22:33.900 100 d=0:1:46
16:22:34.326 GET http://www.google-analytics.com/collect [HTTP/1.1 200 OK 80ms]
16:22:34.734 GET https://apis.google.com/_/scs/abc-static/_/js/k=gapi.gapi.en.DpbM-BtxMjs.O/m=gapi_iframes,googleapis_client,plusone/rt=j/sv=1/d=1/ed=1/am=AAQ/rs=AItRSTM3UB5dHgm_CutMnDlOFNkNRC36yg/cb=gapi.loaded_0 [HTTP/2.0 304 Not Modified 234ms]
16:22:34.963 GET http://www.google.com/finance/qs/test [HTTP/1.1 200 OK 424ms]
16:22:35.284 GET http://www.google.com/finance/qs/test [HTTP/1.1 200 OK 408ms]
16:22:35.661 POST http://www.google.com/finance/qs/channel [HTTP/1.1 200 OK 101ms]
16:22:35.824 GET http://www.google.com/finance/qs/channel [HTTP/1.1 200 OK 458ms]
16:22:35.825 POST http://www.google.com/finance/qs/channel [HTTP/1.1 400 Illegal parameters [Cause: null] 457ms]
16:22:36.124 GET http://www.google.com/finance/s/m4C5KaofCbA/images/cleardot.gif [HTTP/1.1 200 OK 187ms]


Aka this POST
http://www.google.com/finance/qs/channel?VER=8&SID=4523C6A660B39D40&RID=87213&AID=0&zx=ko7k7mx02h1d&t=1

Comment 34

[Karl Dubost💡 :karlcow]

Using webkitVisibilityState and wondering if it has an impact on the final rendering.

                window.jstiming.report = function $Gx(a, b, c) {
                    if ("prerender" == document.webkitVisibilityState) {
                        var d = !1,
                            e = function $Fx() {
                                if (!d) {
                                    b ? b.prerender = "1" : b = {
                                        prerender: "1"
                                    };
                                    var k;
                                    "prerender" == document.webkitVisibilityState ? k = !1 : (yu(a, b, c), k = !0);
                                    k && (d = !0, document.removeEventListener("webkitvisibilitychange", e, !1))
                                }
                            };
                        document.addEventListener("webkitvisibilitychange", e, !1);
                        return ""
                    }

Comment 35

[Karl Dubost💡 :karlcow]

With the "User-Agent: iphone" we get the full content. 
There's definitely something related to the UA/Firefox.

Comment 36

[Karl Dubost💡 :karlcow]

I wonder if we should requalify the bug as DSK only.
The mobile version doesn't exist anymore on Blink.

This is working on DSK
http://www.google.com/finance?q=NASDAQ:GOOG

The null error is still happening on Firefox Android.

Comment 37

[Karl Dubost💡 :karlcow]

We should just ask Google to send us the same version they are sending to Chrome.

Requesting 	http://www.google.com/m/finance on Chrome
Redirects to    http://www.google.com/finance

Comment 38

[Karl Dubost💡 :karlcow]

I asked Google to send the desktop version for Firefox Dsk. Which is working on Firefox Android.

Comment 39

[Karl Dubost💡 :karlcow]

So we receive the desktop version like Chrome. Good.

What is less good is that Firefox Android still triggers something which makes the quote area blank.

We could do UA override to send Firefox Desktop UA instead of Android.

Comment 40

[Mike Taylor [:miketaylr]]

With Google Finance @ google.com/finance, we won't be able to use a UA override w/o sending a desktop UA to all of google.com. 

We should be able to do this with the go faster add-on, though.

Comment 41

[Sergiu Logigan [:sergiu]]

Attachment: Screenshot_1.png

I am unable to reproduce this one right now.

Comment 42

[Sergiu Logigan [:sergiu]]

This one appears to be Fixed. Closing it as such.