[Tizen][M34-Merge] Bringup Inputpicker (Color, Date picker)
authorDongjun Kim <djmix.kim@samsung.com>
Fri, 21 Nov 2014 05:52:16 +0000 (14:52 +0900)
committerYoungsoo Choi <kenshin.choi@samsung.com>
Tue, 10 Jul 2018 06:57:09 +0000 (06:57 +0000)
commiteb6a0db314fda67d7b3a1ab12b6c26b515ae2ae7
tree1bf8bd2516f07c3f578e79a2198e923c4ece0f5d
parentb35040a2ff5241c9cf7af0f0692d42df5ba73f4c
[Tizen][M34-Merge] Bringup Inputpicker (Color, Date picker)

Patches for bringup "input-picker" already merged to M40 from M34_beta.
But it does not sufficiently complete. Color picker is still crashed
and date picker is not shown up. And we resolved crash issue using
patch for "crash in color picker", but date picker still have a problem.
This patch for bug fix in color and date picker.
Detail list of bugs and description of solution are below.

1. Crash in color picker.
[Problem] Crash was occured when click the color picker.
"elm_layout_add()" and "elm_datetime_add()" functions are need to
evas object when click the input picker. We sent evas object using
elm_object_parent_widget_get() in latest code and it returned to NULL.
So, elm_ctxpopup_add() was crashed by NULL object.
[Solution] We change to "m_ewkView" instead of
"elm_object_parent_widget_get(m_ewkView)".

2. Showing up black-screen in "date, week, month, time".
[Problem] Control of data picker is consist of next 6 element.
"date, week, month, time, datetime, datime-local"
In previous, it used different logic while creating the popup.
"datetime, datetime-local" does work and it used elm_layout_add()
when creating. "date, week, month, time" doesn't work and
it doen't used elm_layout_add().
[Solution] Change to use same logic in date picker.

3. Always set to "1900-01-01" in date picker.
[Problem] Parsing logic for input parameter is completely wrong
in date picker. It always set to "1900-01-01" regardless of paramter.
[Solution] Implement parsing logic for input parameter.

4. Clean-up the code.
- 80 characters per line.
- Change function name.
- Remove unused code.

Original patches:
1) fixup! fix for color picker crash by Robert Jadczak
http://165.213.202.130:8080/#/c/68840/
2) Fixed displaying HTML5 control popups by Piotr SzawdyƄski
http://165.213.202.130:8080/#/c/69160/
3) Correcting InputPicker implementation by Lukasz Krok
http://165.213.202.130:8080/#/c/70036

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8765
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I47c0bad9e770f1b1eead43fc245c9a2eeb80335c
Signed-off-by: Dongjun Kim <djmix.kim@samsung.com>
tizen_src/impl/browser/inputpicker/InputPicker.cc
tizen_src/impl/browser/inputpicker/InputPicker.h
tizen_src/impl/eweb_view.cc
tizen_src/impl/eweb_view.h
tizen_src/impl/theme/themeChromiumTizen.css
tizen_src/impl/web_contents_delegate_efl.cc