New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 10 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Aug 2012
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug



Sign in to add a comment
link

Issue 141018: Projectmanager.com application causes Flash to hang

Reported by jeffreyc@google.com, Aug 7 2012 Project Member

Issue description

Pasting from email with subject line "Fwd: Urgent Issue - with Chrome":


"""
Since this update released a few days ago, we have been unable to run up our flash application as it crashes flash at some point in the flash initialization. We have 9,000 paying customers (like {redacted}) and most are using Chrome – meaning that they can’t access their tasks as shown below, it simply locks up the web page:

A plug-in (Shockwave Flash) isn't responding.  [Stop plug-in]

Could you please put me in touch with your engineers to supply a debug version of the Pepper Flash player so we can see what is causing this issue? Alternatively, they can login to https://secure.projectmanager.com with the credentials (User: {redacted}, Pwd: {redacted}) and click on the “Sample” project above to see the issue.
 
The Pepper Flash plugin version details are:
·         Version:  11.3.31.222

·         Location: \AppData\Local\Google\Chrome\Application\21.0.1180.60\PepperFlash\pepflashplayer.dll

 
Customers are quickly considering cancelling their accounts, so if you could please put me in touch with the right people to help fix the issue I would greatly appreciate it.
"""
 

Comment 1 by jeffreyc@google.com, Aug 7 2012

Owner: viettrungluu@chromium.org
Status: Assigned
assigning to Trung as per Karen's instructions.

Comment 2 by bung...@gmail.com, Aug 8 2012

Comment 3 by bung...@gmail.com, Aug 8 2012

We have found a very specific step into reproducing this issue. It seems to be system timezone dependent. 

The page to visit to repro is:
https://alpha.projectmanager.com/scriptsource/ganttchart.html

Set timezone of system to US or AU and visit - works ok.
Set timezone of system to NZ - hangs

Now Im wondering if this is an issue with the pepper API or the updated version of the flash player that is now being used with the latest stable of chrome. Ie is this error occuring in Google's side or Adobe's side

Comment 4 by bung...@gmail.com, Aug 8 2012

We have found a very specific step into reproducing this issue. It seems to be system timezone dependent. 

The page to visit to repro is:
https://alpha.projectmanager.com/scriptsource/ganttchart.html

Set timezone of system to US or AU and visit - works ok.
Set timezone of system to NZ - hangs

Now Im wondering if this is an issue with the pepper API or the updated version of the flash player that is now being used with the latest stable of chrome. Ie is this error occuring in Google's side or Adobe's side

Comment 5 by jsc...@chromium.org, Aug 9 2012

Okay, setting my timezone to Auckland, NZ worked. So, now we have repro.

Comment 6 by jsc...@chromium.org, Aug 9 2012

Owner: jsc...@chromium.org
This works in Pepper Flash if I disable the sandbox. So, I expect Flash is touching something locale-related that either needs to be warmed up or cached. I'm probably the right person to fix it, but I almost certainly won't be able to get to it until Monday.

Comment 7 by ihf@chromium.org, Aug 9 2012

 Issue 140763  has been merged into this issue.

Comment 8 by bung...@gmail.com, Aug 9 2012

Great thanks guys, good work! This issue is affects lots of our accounts so a rapid resolution would be beneficial.

Comment 9 by jsc...@chromium.org, Aug 9 2012

Weird thing is that it seems to be a temporary hang. If I leave it be eventually everything seems to load.

Comment 10 by craig.ni...@gmail.com, Aug 9 2012

Sometimes it does load but the player is still broken if we try to interact with the UI it is still broken.

Comment 11 by craig.ni...@gmail.com, Aug 9 2012

Can you please give me an idea of when this fix will be rolled out? If you start work on the problem on Monday - does that indicate a timeline for the resolution?

Comment 12 by jeffreyc@google.com, Aug 9 2012

It's hard to say until Justin has a chance to sit down and sift through the code, but rest assured as soon as we have a fix, we will work to auto-update all of Chrome's users as quickly as we can (and as responsibly as we can, since each new build requires significant testing).

Comment 13 by craig.ni...@gmail.com, Aug 10 2012

Thanks for the update Jeffrey, much appreciated. If there is anyway we can help (testing?) then please let us know.

Comment 14 by jeffreyc@google.com, Aug 10 2012

Once Justin has a potential fix checked into trunk, we will probably want you to test it using the "Chrome Canary" channel, which can be downloaded and installed separately without affecting your existing Chrome installation: https://tools.google.com/dlpage/chromesxs/

(note: Canary should never be used for normal day-to-day browsing, as it's extremely bleeding-edge and basically used just for testing purposes.)

Comment 15 by viettrungluu@chromium.org, Aug 10 2012

Owner: viettrungluu@chromium.org
Status: Started
Yoink.

I believe it's not a sandbox issue, but that we incorrectly convert between UTC and local time in some cases (it's totally Brett's fault).

Comment 16 by bung...@gmail.com, Aug 11 2012

Given the number of Date objects we use (upon serialization from JSON to AS3 Date objects) that would slow things down.
I checked the Tamarin engine and it uses system dates. Dows the flash player proxy through the browser for date objects when a plugin?

Comment 17 by viettrungluu@chromium.org, Aug 11 2012

@16: It only needs to go to the browser when it needs to convert a UTC time to local time, or vice versa.

The problem in question seems to happen when it tries to convert a local time around the DST transition to UTC. (I thought I saw it trying to convert local times that don't exist -- i.e., in the transition *to* DST, moving clocks forward, there's an hour of local time that doesn't exist.)

Pepper Flash gets something wrong (or different, at least), and this seems to lead to an infinite loop or similar (with it doing conversions over and over). (Not having the AS3 code, I can't really tell what it's doing.)

Comment 18 by w...@launchpadrecruits.com, Aug 13 2012

Is there any update on  the timing of this. Our application has been broken by this issue -> I'm losing customers!!

Comment 19 by jeffreyc@google.com, Aug 14 2012

Cc: jeffreyc@chromium.org

Comment 20 by craig.ni...@gmail.com, Aug 16 2012

Hi Guys

Just a nudge on this one, we are still affected by this and we are having to move the users in the affected time zones on to FireFox/IE which is not that desirable for us.

Please let us know when we can test on Canary.

Comment 21 by viettrungluu@chromium.org, Aug 16 2012

Labels: Action-FeedbackNeeded
Could you check the current Chrome Canary (with Pepper Flash 11.3.31.318+)? I believe it should be fixed. If it looks okay, I'll try to get it in for the next Chrome 21 update. Thanks.

Comment 22 by craig.ni...@gmail.com, Aug 16 2012

Thanks for the update, we will test it immediately and let you know how we get on.

Comment 23 by craig.ni...@gmail.com, Aug 17 2012

Hi. We have tested ProjectManager.com on Canary and all the time zones that were failing and all work well now.

Thumbs up for a roll out from us.

Comment 24 by jeffreyc@google.com, Aug 17 2012

hooray!

Trung, how complex/risky is the fix, and it is in Chrome code, or just Flash code? (Would it be reasonable to request a merge to M21?)

Comment 25 by viettrungluu@chromium.org, Aug 17 2012

Status: Fixed
@jeffreyc: I already merged it. Hopefully the M21 Beta looks okay.

Comment 26 by craig.ni...@gmail.com, Aug 20 2012

Would you like us to test this on the latest beta??

Comment 27 by jeffreyc@google.com, Aug 20 2012

Yes please - specifically, version 21.0.1180.81.

(http://googlechromereleases.blogspot.com/2012/08/beta-channel-update_17.html)

thanks

Comment 28 by craig.ni...@gmail.com, Aug 21 2012

@jeffreyc: we have tested that version and we are happy that our ticket has been addressed.

Comment 29 by jeffreyc@google.com, Aug 21 2012

Status: Verified
excellent; this will be pushed to the Stable channel soon.

Comment 30 by craig.ni...@gmail.com, Aug 23 2012

@jeffreyc: any timeline on the update?

Comment 31 by jeffreyc@google.com, Aug 23 2012

I think the fix should have been in this Stable update: http://googlechromereleases.blogspot.com/2012/08/stable-channel-update_21.html

Comment 32 by dangwyn...@gmail.com, Oct 1 2012

Hi,

We've had numerous customers reporting issues with times displaying incorrectly on our site. Our timer code hasn't been changed for over a year. Customers are reporting that times are appearing 60 minutes out, all users reporting this have been using Chrome with Pepper Flash player, disabling the pepper player resolves the problem in all cases.

Has the timezone code been touched in last weeks release of Pepper Flash Player?

Comment 33 by bung...@gmail.com, Oct 1 2012

Daylight savings was Saturday-Sunday in NZ (ie 60 mins). Might have something to do with that. 

Im not a chrome dev though.

Comment 34 by robert.o...@equator.co.nz, Oct 2 2012

We also have customers reporting incorrect times showing (1 hour out) in our app since the DST change on Sunday.
If we get customers to disable pepper flash the adobe version works okay.
Problem appears to be when handling a pre-Daylight saving time while current date is post daylight saving.

Comment 35 by robert.o...@equator.co.nz, Oct 2 2012

Possibly related to  Issue 146918 .

Comment 36 by bugdroid1@chromium.org, Mar 9 2013

Project Member
Labels: -Action-FeedbackNeeded Needs-Feedback

Comment 37 by bugdroid1@chromium.org, Mar 10 2013

Project Member
Labels: -Area-UI -Feature-Flash -Feature-Plugins-Pepper -Feature-Plugins -Mstone-21 Cr-Content-Plugins M-21 Cr-UI Cr-Content-Plugins-Flash Cr-Content-Plugins-Pepper

Comment 38 by bugdroid1@chromium.org, Apr 6 2013

Project Member
Labels: Cr-Blink

Comment 39 by bugdroid1@chromium.org, Apr 6 2013

Project Member
Labels: -Cr-Content-Plugins-Flash Cr-Internals-Plugins-Flash

Comment 40 by bugdroid1@chromium.org, Apr 6 2013

Project Member
Labels: -Cr-Content-Plugins Cr-Internals-Plugins

Comment 41 by bugdroid1@chromium.org, Apr 6 2013

Project Member
Labels: -Cr-Content-Plugins-Pepper Cr-Internals-Plugins-Pepper

Comment 42 by laforge@google.com, Jul 24 2013

Cc: -jeffreyc@chromium.org

Sign in to add a comment