Bug 147725 – [EFL] Use the non-overlay scrollbar

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit EFL

[Hunseop Jeong]

I think non-overlay scrollbar looks better than the overlay scrollbar and other ports do that.
But I turned on the overlay scrollbar to draw the scrollbar when I uploaded the patch(http://trac.webkit.org/changeset/186829).
Because non-overlay scrollbar has a problem(https://bugs.webkit.org/show_bug.cgi?id=140243).
If /b/140243 is fixed, we can show the non-overlay scrollbar with this patch.

[Hunseop Jeong]

Created attachment 258358 [details]
Patch

[Ryuan Choi]

(In reply to comment #0)
> I think non-overlay scrollbar looks better than the overlay scrollbar and
> other ports do that.
> But I turned on the overlay scrollbar to draw the scrollbar when I uploaded
> the patch(http://trac.webkit.org/changeset/186829).
> Because non-overlay scrollbar has a
> problem(https://bugs.webkit.org/show_bug.cgi?id=140243).
> If /b/140243 is fixed, we can show the non-overlay scrollbar with this patch.

I think that it should be an optional via theme.

[Hunseop Jeong]

Created attachment 258359 [details]
Patch

[Hunseop Jeong]

(In reply to comment #2)
> (In reply to comment #0)
> > I think non-overlay scrollbar looks better than the overlay scrollbar and
> > other ports do that.
> > But I turned on the overlay scrollbar to draw the scrollbar when I uploaded
> > the patch(http://trac.webkit.org/changeset/186829).
> > Because non-overlay scrollbar has a
> > problem(https://bugs.webkit.org/show_bug.cgi?id=140243).
> > If /b/140243 is fixed, we can show the non-overlay scrollbar with this patch.
> 
> I think that it should be an optional via theme.

I changed the condition to use the setting value.

[WebKit Commit Bot]

Comment on attachment 258359 [details]
Patch

Clearing flags on attachment: 258359

Committed r188206: <http://trac.webkit.org/changeset/188206>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Gyuyoung Kim]

(In reply to comment #5)
> Comment on attachment 258359 [details]
> Patch
> 
> Clearing flags on attachment: 258359
> 
> Committed r188206: <http://trac.webkit.org/changeset/188206>


EFL API test has been broken since r188206.

ERR<5605>:ecore_evas modules/ecore_evas/engines/x/ecore_evas_x.c:4493 ecore_evas_gl_x11_options_new_internal() evas_engine_info_set() init engine 'opengl_x11' failed.
ERR<5605>:evas_main lib/evas/canvas/evas_gl.c:138 evas_gl_new() Evas GL engine not available.
ERR<5605>: ../../Source/WebKit2/UIProcess/API/efl/ewk_view.cpp:54 toEwkViewChecked() safety check failed: evasObject == NULL
CRI<5605>: ../../Source/WebKit2/UIProcess/API/efl/ewk_view.cpp:767 ewk_view_contents_size_get() no private data for object (nil)
../../Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_view.cpp:1208: Failure
Value of: contentsWidth
  Actual: 790
Expected: environment->defaultWidth()
Which is: 800