Summary: | Fix EFL Build errors since r166975. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Hyowon Kim <hw1008.kim> | ||||||
Component: | WebKit EFL | Assignee: | Hyowon Kim <hw1008.kim> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bunhere, cdumez, cmarcelo, commit-queue, gyuyoung.kim, lucas.de.marchi, luiz, noam, rakuco, sergio, zeno | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Linux | ||||||||
Attachments: |
|
Description
Hyowon Kim
2014-04-08 20:18:52 PDT
Created attachment 228930 [details]
Patch
Comment on attachment 228930 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=228930&action=review > Source/WebKit2/PlatformEfl.cmake:189 > + WebProcess/WebPage/PageOverlayController.cpp I think that this is not EFL specific Comment on attachment 228930 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=228930&action=review > Source/WebKit2/ChangeLog:3 > + EFL Build fix since r166956 I don't know why r166956 is involved in this build break. It looks the r166956 make bmalloc. http://trac.webkit.org/changeset/166956 (In reply to comment #3) > (From update of attachment 228930 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=228930&action=review > > > Source/WebKit2/ChangeLog:3 > > + EFL Build fix since r166956 > > I don't know why r166956 is involved in this build break. > > It looks the r166956 make bmalloc. > > http://trac.webkit.org/changeset/166956 Sorry r166975. Created attachment 228932 [details]
Patch
I've filed a new bug 131425 to add additional implementation for page overlay. Comment on attachment 228932 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=228932&action=review > Source/WebKit2/WebProcess/WebPage/TapHighlightController.cpp:73 > + m_webPage->uninstallPageOverlay(m_overlay, PageOverlay::FadeMode::Fade); BTW, should we modify this line ? Comment on attachment 228932 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=228932&action=review > Source/WebKit2/WebProcess/WebPage/TapHighlightController.cpp:73 > + m_webPage->uninstallPageOverlay(m_overlay, PageOverlay::FadeMode::Fade); BTW, should we modify this line ? (In reply to comment #8) > (From update of attachment 228932 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=228932&action=review > > > Source/WebKit2/WebProcess/WebPage/TapHighlightController.cpp:73 > > + m_webPage->uninstallPageOverlay(m_overlay, PageOverlay::FadeMode::Fade); > > BTW, should we modify this line ? Yes, the second parameter was changed to enum class FadeMode. Comment on attachment 228932 [details] Patch Clearing flags on attachment: 228932 Committed r167002: <http://trac.webkit.org/changeset/167002> All reviewed patches have been landed. Closing bug. |