This looks like a regression from 1040187. http://mxr.mozilla.org/mozilla-central/source/gfx/layers/composite/TextureHost.cpp#364 mFirstSource->GetUpdateSerial() == mUpdateSerial will never be true since we've just bumped mUpdateSerial. The mPartialUpdate part of the condition won't ever be true either since this is the only place we set it to true.

Comment on attachment 8480927 [details] [diff] [review] fix-partial-upload Review of attachment 8480927 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, but needs some comments.

Comment on attachment 8480927 [details] [diff] [review] fix-partial-upload Approval Request Comment [Feature/regressing bug #]: OMTC [User impact if declined]: Drawing artifacts [Describe test coverage new/current, TBPL]: Nightly & Aurora [Risks and why]: Required if we uplift the fix for bug 1040187 [String/UUID change made/needed]: None

Comment on attachment 8480927 [details] [diff] [review] fix-partial-upload beta+. Approval already granted for bug 1040187. These changes will ship in beta2.

Made a silly rebase error. Backed out and re-landed. https://hg.mozilla.org/releases/mozilla-beta/rev/bca701646487