Skip to content
This repository has been archived by the owner on Aug 20, 2020. It is now read-only.

Fixed screen displayed abnormally when playing some size of html5 mp4 vi... #268

Closed
wants to merge 1 commit into from

Conversation

qjia7
Copy link
Contributor

@qjia7 qjia7 commented Aug 19, 2014

...deos

This patch is cherry picked from Qing's PR
crosswalk-project/chromium-crosswalk#166

Bug = https://crosswalk-project.org/jira/browse/XWALK-2265
#260 can also fix this problem. However, after talking with Changbin, it seems that #260 can't be merged in a short time. So, add this patch to fix the current implementation's defect.

@tiagovignatti
Copy link
Contributor

ok, I'm fine on having this workaround in oz-wl repository. But can you amend a nice commentary in the code changed there, similar as the information you put in the commit message? I think it may help other developers that eventually will bump into that snip of code. Thank you.

@qjia7
Copy link
Contributor Author

qjia7 commented Aug 20, 2014

@tiagovignatti Done. PTAL

@tiagovignatti
Copy link
Contributor

LGTM. I'm deferring to @kalyankondapally to merge it now. Thank you.

@kalyankondapally
Copy link
Contributor

@qjia7 Merged thanks.

Next time, lease make sure the commit message dont exceed 80 chars.

@kalyankondapally kalyankondapally added this to the Harvest milestone Aug 21, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants