Bug 80211 - [EFL] Evas_GL initialization and destruction in GraphicsContext3DPrivate
Summary: [EFL] Evas_GL initialization and destruction in GraphicsContext3DPrivate
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 62961
Blocks: 79766
  Show dependency treegraph
 
Reported: 2012-03-03 01:51 PST by Hyowon Kim
Modified: 2012-03-04 19:15 PST (History)
5 users (show)

See Also:


Attachments
Patch (12.03 KB, patch)
2012-03-03 01:55 PST, Hyowon Kim
no flags Details | Formatted Diff | Diff
proposed patch (6.66 KB, patch)
2012-03-03 02:04 PST, Hyowon Kim
noam: review-
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
modified patch (7.07 KB, patch)
2012-03-04 00:59 PST, Hyowon Kim
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hyowon Kim 2012-03-03 01:51:55 PST
The last patch for GC3DPrivate implementation, which contains initialize() and the destructor function using Evas_GL APIs.
Comment 1 Hyowon Kim 2012-03-03 01:55:20 PST
Created attachment 129998 [details]
Patch
Comment 2 Hyowon Kim 2012-03-03 02:04:08 PST
Created attachment 129999 [details]
proposed patch
Comment 3 WebKit Review Bot 2012-03-03 04:00:16 PST
Comment on attachment 129999 [details]
proposed patch

Attachment 129999 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/11799322

New failing tests:
editing/selection/select-line-break-with-opposite-directionality.html
Comment 4 Noam Rosenthal 2012-03-03 07:46:56 PST
Comment on attachment 129999 [details]
proposed patch

View in context: https://bugs.webkit.org/attachment.cgi?id=129999&action=review

> Source/WebCore/platform/graphics/efl/GraphicsContext3DPrivate.cpp:58
> +    if (m_evasGL) {

Style: early-return here.

> Source/WebCore/platform/graphics/efl/GraphicsContext3DPrivate.cpp:87
> +    m_context = evas_gl_context_create(m_evasGL, 0);

What does 0 stand for? comment.

> Source/WebCore/platform/graphics/efl/GraphicsContext3DPrivate.cpp:102
> +    int x = 0, y = 0, width = 1, height = 1;

separate lines.
Comment 5 Hyowon Kim 2012-03-04 00:59:50 PST
Created attachment 130018 [details]
modified patch

I've added some codes for context sharing between AC and WebGL.
Comment 6 WebKit Review Bot 2012-03-04 19:15:32 PST
Comment on attachment 130018 [details]
modified patch

Clearing flags on attachment: 130018

Committed r109696: <http://trac.webkit.org/changeset/109696>
Comment 7 WebKit Review Bot 2012-03-04 19:15:37 PST
All reviewed patches have been landed.  Closing bug.