[webkit-efl] Scope of 'ewk_util.h'
Kangil Han
kangil.han at samsung.com
Tue Jul 12 21:01:49 PDT 2011
Dear Gyuyoung,
Sure, I will make a patch.
Thanks and regards,
Kangil
------- Original Message -------
Sender : 김규영<gyuyoung.kim at samsung.com> E4(선임)/선임/S/W Platform Lab(DMC연)/삼성전자
Date : 2011-07-13 09:40 (GMT+09:00)
Title : RE: [webkit-efl] Scope of 'ewk_util.h'
Hello Rafael,
Long time no see. How are you? :-)
I understand role of the function. I also think we don't need to add the ewk_util.h to EWebKit_HEADERS yet.
IMO, we can add it to EWebkit_HEADER again when we need to open the function.
Kangil,
Could you make a patch for this ? You can remove it from Source/WebKit/efl/CMakeListEfl.txt.
Please make reference to the file.
217 FILE(GLOB EWebKit_HEADERS "${CMAKE_CURRENT_SOURCE_DIR}/efl/ewk/*.h")
218 LIST(REMOVE_ITEM EWebKit_HEADERS "${CMAKE_CURRENT_SOURCE_DIR}/efl/ewk/ewk_private.h")
219 LIST(REMOVE_ITEM EWebKit_HEADERS "${CMAKE_CURRENT_SOURCE_DIR}/efl/ewk/ewk_tiled_private.h")
220 LIST(REMOVE_ITEM EWebKit_HEADERS "${CMAKE_CURRENT_SOURCE_DIR}/efl/ewk/ewk_tiled_backing_store.h")
221 LIST(REMOVE_ITEM EWebKit_HEADERS "${CMAKE_CURRENT_SOURCE_DIR}/efl/ewk/ewk_tiled_matrix.h")
222 LIST(REMOVE_ITEM EWebKit_HEADERS "${CMAKE_CURRENT_SOURCE_DIR}/efl/ewk/ewk_tiled_model.h")
- Gyuyoung
> -----Original Message-----
> From: Rafael Antognolli [mailto:antognolli at profusion.mobi]
> Sent: Tuesday, July 12, 2011 10:12 PM
> To: Gyuyoung Kim
> Cc: ryuan.choi at samsung.com; 한강일; webkit-efl at lists.webkit.org
> Subject: Re: [webkit-efl] Scope of 'ewk_util.h'
>
> Hello Gyuyoung and Ryuan!
>
> Long time no see! :-)
>
> On Tue, Jul 12, 2011 at 4:05 AM, Gyuyoung Kim <gyuyoung.kim at samsung.com>
> wrote:
> > It looks role of “ewk_util_image_from_cairo_surface_add()” APIs is to
> get
> > image from cairo surface.
> > I also don’t know why the APIs was created. I think Profusion guys may
> know
> > the reason.
>
> The reason is that ewk_history.cpp and ewk_settings.cpp use this
> function to create an Evas_Object of type image from the cairo
> surface. This is useful when creating a thumbnail, or an icon for that
> page. Look for this function on these files and you will understand.
>
> > Anyway, I think we don’t need to open ewk_util.h unless ewk_util has
> > valuable APIs.
>
> I agree.
>
> > - Gyuyoung
> >
> > From: webkit-efl-bounces at lists.webkit.org
> > [mailto:webkit-efl-bounces at lists.webkit.org] On Behalf Of Ryuan Choi
> > Sent: Tuesday, July 12, 2011 4:00 PM
> > To: 한강일
> > Cc: webkit-efl at lists.webkit.org
> > Subject: Re: [webkit-efl] Scope of 'ewk_util.h'
> >
> >
> >
> >> Otherwise, ‘EAPI’ should be added in front of return type.
> >
> >
> >
> > ewk_util.h is not public now.
> >
> > I saw bugs to enable it as public, but I am not sure.
> >
> >
> >
> > Does we have any reasons to make EAPI?
> >
> > IMO, we'd better to remove it from EWebKit_HEADERS.
>
> Not sure now why this file is in EWebKit_HEADERS but I think removing
> it from there is probably a good idea.
>
> Regards,
> --
> Rafael Antognolli
> ProFUSION embedded systems
> http://profusion.mobi
<p> </p><p> </p>
More information about the webkit-efl
mailing list