Bug 100731

Summary: [EFL][WK2] Let Ecore_Evas own the cursor object
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit EFLAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: gyuyoung.kim, kenneth, laszlo.gombos, lucas.de.marchi, mikhail.pozdnyakov, rakuco, ryuan.choi, tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Chris Dumez
Reported 2012-10-30 00:34:11 PDT
We currently have EwkViewImpl own the cursor Evas Object although Ecore_Evas can take of it by itself. When calling ecore_evas_object_cursor_set(), the Ecore_Evas code will free the currently set cursor object (calling evas_object_del) before setting the new one. Therefore, we don't need to do that by ourselves and do reference counting.
Attachments
Patch (3.99 KB, patch)
2012-10-30 00:53 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2012-10-30 00:53:51 PDT
Created attachment 171386 [details] Patch
WebKit Review Bot
Comment 2 2012-10-30 01:54:49 PDT
Comment on attachment 171386 [details] Patch Clearing flags on attachment: 171386 Committed r132880: <http://trac.webkit.org/changeset/132880>
WebKit Review Bot
Comment 3 2012-10-30 01:54:53 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.