RESOLVED INVALID Bug 83248
[BlackBerry] Upstream the BlackBerry change to rendering/RenderPart.cpp 
https://bugs.webkit.org/show_bug.cgi?id=83248
Summary [BlackBerry] Upstream the BlackBerry change to rendering/RenderPart.cpp
Leo Yang
Reported 2012-04-04 22:59:07 PDT
The blackberry porting is using a different way to accelerate plugins.
Attachments
Patch (2.23 KB, patch)
2012-04-04 23:04 PDT, Leo Yang 		no flags
DetailsFormatted DiffDiff
Patch v2 (2.39 KB, patch)
2012-04-05 18:48 PDT, Leo Yang 		tonikitoo: review-
tonikitoo: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Leo Yang
Comment 1 2012-04-04 23:04:48 PDT
Created attachment 135759 [details] Patch
Rob Buis
Comment 2 2012-04-05 03:53:06 PDT
Comment on attachment 135759 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=135759&action=review > Source/WebCore/rendering/RenderPart.cpp:86 > +#endif How about: if (widget() && widget()->isPluginViewBase() && static_cast<PluginViewBase*>(widget())->platformLayer()) #if !PLATFORM(BLACKBERRY) return true; #else return false; #endif Then even if the code after that changes, we keep doing the right thing for plugins.
Leo Yang
Comment 3 2012-04-05 18:48:35 PDT
Created attachment 135965 [details] Patch v2 Revised to follow Rob's suggestion
Antonio Gomes
Comment 4 2012-04-10 08:28:42 PDT
Comment on attachment 135965 [details] Patch v2 Sorry, Leo, but I do not want to add a #if plat(bb) here like this. I would suggest: - make a helper with a meaningful name, and add if's there. Maybe even a static local function. - add a setting, where bb is the only platform that default returns false. also it would be good to mention where in the code blackberry port we HW accelerate plugin layers, etc.
Note You need to log in before you can comment on or make changes to this bug.