Bug 147626 - [EFL] It doesn't work aircraft view in Naver map
Summary: [EFL] It doesn't work aircraft view in Naver map
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: http://map.naver.com
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-04 04:58 PDT by Jincheol Jo
Modified: 2015-08-09 17:25 PDT (History)
2 users (show)

See Also:

Attachments
Patch (2.97 KB, patch)
2015-08-04 20:16 PDT, Jincheol Jo 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.