[Webkit-unassigned] [Bug 61850] [EFL] Add MiniBrowserEfl.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 1 04:21:05 PDT 2011


--- Comment #3 from Lucas De Marchi <demarchi at webkit.org>  2011-06-01 04:21:05 PST ---
(From update of attachment 95572)
View in context: https://bugs.webkit.org/attachment.cgi?id=95572&action=review

Once WebKit2 is compiled in EFL port, I think this can enter. I agree with Gyuyoung that the cmake file should be in efl directory. But we can do a bit better and split it in a generic CMakeLists.txt file and another efl/CMakeListsEfl.txt. This way other ports that use cmake can profit from this work too.

> Tools/MiniBrowser/CMakeListsEfl.txt:15
> +    ${WEBKIT2_DIR}/UIProcess/API/efl
> +    ${WEBKIT2_DIR}

Since we don't support webkit2, are you sure you need this?

> Tools/MiniBrowser/efl/main.c:55
> +static int browserCreate(const char *url, const char *engine)
> +{
> +    MiniBrowser *app = (MiniBrowser*) malloc(sizeof(MiniBrowser));
> +
> +    app->ee = ecore_evas_software_x11_new(NULL, 0, 0, 0, DEFAULT_WIDTH, DEFAULT_HEIGHT);

Could you use ecore_evas_new() here instead?

> Tools/MiniBrowser/efl/main.c:56
> +    ecore_evas_title_set(app->ee, "Samsung MiniBrowser");

I think it's not good to let Samsung here. WebKit would be a better name.

> Tools/MiniBrowser/efl/main.c:71
> +    /* Create webview */
> +    app->browser = ewk_view_add(app->evas, WKContextGetSharedProcessContext(), 0);

It seems you are indeed using webkit2. I'm wondering what's happening since we don't have support to webkit2 yet.

> Tools/MiniBrowser/efl/main.c:97
> +    browserCreate(url, engine);
> +
> +    ecore_event_handler_add(ECORE_EVENT_SIGNAL_EXIT, mainSignalExit, NULL);

Since in this file we are following EFL style, I think it's better not to use camelCase for function names.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list