WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
147626
[EFL] It doesn't work aircraft view in Naver map
https://bugs.webkit.org/show_bug.cgi?id=147626
Summary
[EFL] It doesn't work aircraft view in Naver map
Jincheol Jo
Reported
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.
Attachments
Patch
(2.97 KB, patch)
2015-08-04 20:16 PDT
,
Jincheol Jo
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Jincheol Jo
Comment 1
2015-08-04 20:16:09 PDT
Created
attachment 258260
[details]
Patch
Gyuyoung Kim
Comment 2
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 ?
Jincheol Jo
Comment 3
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.
Jincheol Jo
Comment 4
2015-08-04 22:41:04 PDT
Also TOUCH_EVENTS is disabled in Minibrowser OSX.
Gyuyoung Kim
Comment 5
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.
Gyuyoung Kim
Comment 6
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.
Jincheol Jo
Comment 7
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.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug