Bug 136410 - REGRESSION (r168217): Images are cropped out during animation at jetblue.com
Summary: REGRESSION (r168217): Images are cropped out during animation at jetblue.com
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P1 Normal
Assignee: Simon Fraser (smfr)
URL: http://www.jetblue.com/flying-on-jetb...
Keywords: InRadar, Regression
Depends on: 139632
Blocks:
  Show dependency tree / graph
 
Reported: 2014-08-30 23:59 PDT by mitz
Modified: 2014-12-15 12:24 PST (History)
6 users (show)

See Also:

Attachments
Patch (13.85 KB, patch)
2014-12-12 14:23 PST, Simon Fraser (smfr) 		no flags Details | Formatted Diff | Diff
Patch (2.04 KB, patch)
2014-12-12 15:33 PST, Simon Fraser (smfr) 		no flags Details | Formatted Diff | Diff
Patch (15.94 KB, patch)
2014-12-15 12:03 PST, Simon Fraser (smfr) 		thorton: review+
Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2014-08-30 23:59:56 PDT 
To reproduce: navigate to the URL and scroll down to the “Sweet seats” section, and click one of the horizontal arrow buttons. As the content scrolls in, the image is partially cropped out during the transition, and the missing part only pops in at the end.

This was caused by <http://trac.webkit.org/r168217>, the fix for bug 132507.
Comment 1 mitz 2014-08-31 00:00:45 PDT 
<rdar://problem/18188533>
Comment 2 Simon Fraser (smfr) 2014-12-12 14:23:13 PST 
Created attachment 243215 [details]
Patch
Comment 3 Simon Fraser (smfr) 2014-12-12 14:52:40 PST 
https://trac.webkit.org/r177238
Comment 4 Simon Fraser (smfr) 2014-12-12 15:33:48 PST 
Reopening to attach new patch.
Comment 5 Simon Fraser (smfr) 2014-12-12 15:33:49 PST 
Created attachment 243222 [details]
Patch
Comment 6 Simon Fraser (smfr) 2014-12-12 15:45:24 PST 
https://trac.webkit.org/r177244
Comment 7 Alexey Proskuryakov 2014-12-14 18:44:19 PST 
r177238 introduced memory corruption on a regression test. Will roll out.

run-webkit-tests -g transitions/transition-end-event-destroy-iframe.html

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   com.apple.WebCore             	0x000000010760af00 WebCore::AnimationControllerPrivate::endAnimationUpdate() + 16
1   com.apple.WebCore             	0x00000001076186b8 WebCore::AnimationPrivateUpdateBlock::~AnimationPrivateUpdateBlock() + 24
2   com.apple.WebCore             	0x000000010760d065 WebCore::AnimationPrivateUpdateBlock::~AnimationPrivateUpdateBlock() + 21
3   com.apple.WebCore             	0x00000001076098b4 WebCore::AnimationControllerPrivate::animationTimerFired() + 68
4   com.apple.WebCore             	0x0000000107616303 std::__1::__function::__func<std::__1::__bind<void (WebCore::AnimationControllerPrivate::*&)(), WebCore::AnimationControllerPrivate*>, std::__1::allocator<std::__1::__bind<void (WebCore::AnimationControllerPrivate::*&)(), WebCore::AnimationControllerPrivate*> >, void ()>::operator()() + 259
5   com.apple.WebCore             	0x00000001075dcb4a std::__1::function<void ()>::operator()() const + 26
6   com.apple.WebCore             	0x00000001075dcafc WebCore::Timer::fired() + 28
Comment 8 WebKit Commit Bot 2014-12-14 18:46:04 PST 
Re-opened since this is blocked by bug 139632
Comment 9 Simon Fraser (smfr) 2014-12-15 12:03:39 PST 
Created attachment 243303 [details]
Patch
Comment 10 Simon Fraser (smfr) 2014-12-15 12:24:32 PST 
Re-landed in https://trac.webkit.org/r177302