Bug 147626

Summary:

[EFL] It doesn't work aircraft view in Naver map

Product:

WebKit

Reporter:

Jincheol Jo <jincheol.jo>

Component:

WebKit EFL

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED WONTFIX

Severity:

Normal

CC:

gyuyoung.kim, lucas.de.marchi

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

URL:

http://map.naver.com

Attachments:

Description	Flags
Patch	none

Description Jincheol Jo 2015-08-04 04:58:11 PDT

Reproducing Step:
1. Load http://map.naver.com
2. Click aircraft icon in right side bar and then you can see aircraft icons on map area.
3. Click this.

Expected Behavior : It loads aircraft view. If Flash Player is not available, it display a message.

EFL Mini Browser : nothing.

Comment 1 Jincheol Jo 2015-08-04 20:16:09 PDT

Created attachment 258260 [details]
Patch

Comment 2 Gyuyoung Kim 2015-08-04 21:45:49 PDT

Comment on attachment 258260 [details]
Patch

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

> Source/cmake/OptionsEfl.cmake:120
> +WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_TOUCH_EVENTS PUBLIC OFF)

Are there many problems when we enables touch event on MiniBrowser ?

Comment 3 Jincheol Jo 2015-08-04 22:13:39 PDT

in http://map.daum.net : 
1. We cannot drag in map area.
2. "LoadView" doesn't work.

in http://map.naver.com :
1. We cannot use "aircraftview"
2. "mouseover" is not work.

These sites regard as mobile environment. 
So it doesn't call MouseEvent's addListeners.
  
If Touch device is not present, I think it makes sense to turn off TOUCH_EVENTS.

Comment 4 Jincheol Jo 2015-08-04 22:41:04 PDT

Also TOUCH_EVENTS is disabled in Minibrowser OSX.

Comment 5 Gyuyoung Kim 2015-08-04 22:50:09 PDT

(In reply to comment #3)
> in http://map.daum.net : 
> 1. We cannot drag in map area.
> 2. "LoadView" doesn't work.
> 
> in http://map.naver.com :
> 1. We cannot use "aircraftview"
> 2. "mouseover" is not work.
> 
> These sites regard as mobile environment. 
> So it doesn't call MouseEvent's addListeners.
>   
> If Touch device is not present, I think it makes sense to turn off
> TOUCH_EVENTS.

To disable touch event, we have to skip tests regarding touch event too. However I want to continue to test touch event as much as possible on layout test. I wonder if we can disable touch event only for MiniBrowser.

Comment 6 Gyuyoung Kim 2015-08-04 23:02:34 PDT

MiniBrowser is a simple test browser. I object to disable touch event in order to show custom icon on MiniBrowser. Because touch event is very important feature for EFL port, we have to maintain if it works well. If we disable it, it is hard to maintain it.

Comment 7 Jincheol Jo 2015-08-05 00:56:58 PDT

(In reply to comment #6)
> MiniBrowser is a simple test browser. I object to disable touch event in
> order to show custom icon on MiniBrowser. Because touch event is very
> important feature for EFL port, we have to maintain if it works well. If we
> disable it, it is hard to maintain it.

I agree with you.
OK no problem.