Bug 39023 - REGRESSION: Google Analytics broken, reports an error when trying to view a site
Summary: REGRESSION: Google Analytics broken, reports an error when trying to view a site
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac (Intel) OS X 10.6
: P1 Major
Assignee: Nobody
URL:
Keywords: InRadar, Regression
Depends on:
Blocks:
 
Reported: 2010-05-12 13:36 PDT by Ain Tohvri
Modified: 2010-06-16 01:32 PDT (History)
3 users (show)

See Also:


Attachments
Safari crash log (36.05 KB, text/plain)
2010-05-12 13:38 PDT, Ain Tohvri
no flags Details
Error on Google Analytics (61.91 KB, image/png)
2010-05-12 13:40 PDT, Ain Tohvri
no flags Details
Analytics markup and JS errors (21.45 KB, image/png)
2010-05-13 04:36 PDT, Ain Tohvri
no flags Details
Malformed markup of Analytics (22.29 KB, text/html)
2010-05-18 03:06 PDT, Ain Tohvri
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ain Tohvri 2010-05-12 13:36:58 PDT
Steps to reproduce:
1. Log in to Google Analytics
2. Click on a profile to see the sites tracked
3. Click on the site
4. "This page has encountered an error that may prevent it from working correctly." error appears

Remarks and the reason for the bug report:
1. Analytics is not broken on Safari (Version 4.0.5 (6531.22.7)) or Chrome
2. Safari crashed on opening a Web Inspector and refreshing the page (crash log attached). 
3. There were HTML errors on page, namely:
[The HTML that caused this error was generated by a script.] Unmatched </link> encountered.  Ignoring tag. settings:19
<meta> is not allowed inside <div>. Moving <meta> into the <head>. settings:21
Comment 1 Ain Tohvri 2010-05-12 13:38:08 PDT
Created attachment 55893 [details]
Safari crash log

Safari crash log on a refresh with Web Inspector open
Comment 2 Ain Tohvri 2010-05-12 13:40:23 PDT
Created attachment 55894 [details]
Error on Google Analytics

Account page of Google Analytics with the error: "This page has encountered an error that may prevent it from working correctly."
Comment 3 Mark Rowe (bdash) 2010-05-12 14:53:17 PDT
<rdar://problem/7976273>
Comment 4 Ain Tohvri 2010-05-13 04:36:29 PDT
Created attachment 55961 [details]
Analytics markup and JS errors

Web Inspector reveals following HTML and JS errors. 

Attached for Google to note.
Comment 5 Alexey Proskuryakov 2010-05-14 19:58:25 PDT
Please confirm whether you are talking about two completely separate issues.
1. Google Analytics crashes in shipping Safari/WebKit when reloaded with open Web Inspector.
2. Google Analytics works in shipping Safari/WebKit, but not in nightlies.

The former issue has been fixed long ago (r47151).
Comment 6 Alexey Proskuryakov 2010-05-14 19:58:58 PDT
Closed by mistake, reopening.
Comment 7 Ain Tohvri 2010-05-16 04:17:19 PDT
Yes, 2 issues:

1. Analytics does not work with WebKit nightly (r59204)
2. Analytics crashes the shipped Safari on refresh with Web Inspector open (Version 4.0.5 (6531.22.7))
Comment 8 Alexey Proskuryakov 2010-05-16 09:44:37 PDT
Thanks. Let's use this to track #1, because #2 has been fixed long ago, and because tracking two issues in one Bugzilla bug is just confusing.
Comment 9 Ain Tohvri 2010-05-16 11:56:40 PDT
Yes, sorry for confusing this report with something that should have been reported to Apple. I understand the #2 hasn't landed in Safari yet, thus the issue.

#1 is the tough one and my main concern is that it wasn't the behaviour in Safari despite the errors on page and the eventual crash that only happened on a refresh. Thus the report.
Comment 10 Alexey Proskuryakov 2010-05-17 18:31:09 PDT
I couldn't reproduce with r59204 nightly.
Comment 11 Ain Tohvri 2010-05-18 02:52:23 PDT
Reproduced on r59633. Logged out from Analytics and made a full reset that revealed an issue described in bug 39278 so I had to manually remove all cookies. Unfortunately didn't help.
Comment 12 Ain Tohvri 2010-05-18 03:06:11 PDT
Created attachment 56345 [details]
Malformed markup of Analytics

HTML of Analytics' malformed markup, reveals 13 errors against HTML 4.01 Transitional.
Comment 13 Alexey Proskuryakov 2010-05-18 10:23:40 PDT
Can you reproduce any of these problems under a guest Mac OS X account? I'm trying to figure out if there can be something different in your Safari settings. Testing on another machine would be helpful, too.
Comment 14 Naofumi Kagami 2010-06-01 20:35:03 PDT
I am getting issue #1 (1. Analytics does not work with WebKit nightly (r59204)) on a guest account.

To clarify, at least in my case, this bug does not crash Webkit. It just shows the page as in the attachment "Error on Google Analytics" and the page does not work correctly indeed, as the message suggests.

The error occurs regardless of whether the Web Inspector is open or not.

This is indeed a REGRESSION, since the web page works fine in Safari 4.

In my testing, this regression was introduced between r58482 (Apr 29) and r58572 (Apr 30). Nightly r58482 works fine whereas r58572 displays the error message.

In the Web Inspector console, I do get the error regarding the <meta> tag placement (as in line 2 of the attachment "Analytics markup and JS errors"). I do not get any other errors however.



(In reply to comment #13)
> Can you reproduce any of these problems under a guest Mac OS X account? I'm trying to figure out if there can be something different in your Safari settings. Testing on another machine would be helpful, too.
Comment 15 Ain Tohvri 2010-06-07 21:12:15 PDT
Also confirmed for Version 5.0 (6533.16, r60742), so didn't land in Safari 5!
Comment 16 Naofumi Kagami 2010-06-07 23:27:48 PDT
I also can confirm that this bug has made it into the release version of Safari 5. This is very unfortunate.

I have another web site that I am rather sure that is showing a very similar problem.

Steps to reproduce;

1. Go to https://www.google.com/adplanner
2. Log in with your Google account.
3. Page stops loading with the URL at https://www.google.com/adplanner/?pli=1 and a "Loading..." message.

In the web inspector console, we get

"RangeError: Maximum call stack size exceeded.   94AB7A7170526263F9914A30DCDED58C.cache.html:8734"

Unfortunately, since Google has obfuscated their JavaScript, I can't understand what is happening.
Comment 17 Alexey Proskuryakov 2010-06-08 10:20:22 PDT
Let's track Google AdPlanner separately - we don't know if that's the same underlying issue. I filed bug 40306 for it.
Comment 18 Naofumi Kagami 2010-06-15 23:15:08 PDT
At least for me, both the Google Analytics bug and the DoubleClick Ad planner bug are solved on the recent nightlies, after Oliver Hunt applied his fixes.

Thanks
Comment 19 Ain Tohvri 2010-06-16 01:32:48 PDT
I can confirm this working on r61173 + Safari 5 after 10.6.4 update today. Looks like it has landed.