Edje, Elementary: Remove <password=off> tag when password mode is disabled 77/75677/3
authorYoungbok Shin <youngb.shin@samsung.com>
Tue, 21 Jun 2016 02:02:10 +0000 (11:02 +0900)
committerGerrit Code Review <gerrit@review.vlan103.tizen.org>
Tue, 21 Jun 2016 06:19:05 +0000 (23:19 -0700)
commitfda3cf5316696bdda7005f79206f376eeb122c94
tree7a8e22ecd39f45ccdb7be6c0ed458b92e8a5f329
parent3a9c3a5a17bb7ec4ade46657fa7743c1757fa128
Edje, Elementary: Remove <password=off> tag when password mode is disabled

Summary:
When edje_password_show_last option is enabled, the edje_entry uses <password=off>
for showing last character. But, when password mode is disabled by the elm_entry,
<password=off> is remained in the text. It can cause some problems.

Because, there is no way to control password mode by API for the edje_entry.
The elm_entry can't remove <password=off> tag before getting text from the edje_entry.
So, the patch adds edje_object_part_text_hide_visible_password() function and
the elm_entry will use this when elm_layout_theme_apply() is called.
@fix

Test Plan:
1. Run "elementary_test".
2. Show "Entry Password" demo. (Newly added by this patch)
3. Password mode is enabled. Put some text.
4. Click "Show Password" check box to disable password mode.
5. Put more text.
6. Click "Hide Password" check box to enable password mode again.
7. See a character among the text is visible. (without this patch)

Reviewers: tasn, herdsman, cedric, jpeg, thiepha, raster

Reviewed By: raster

Subscribers: Blackmole, z-wony, woohyun

Differential Revision: https://phab.enlightenment.org/D3988

* It includes a code for applying ON_HOLD flag for "Return" key event.
  The code is merged into it by my mistake. But, it also should be merged to fix issues.

Change-Id: I6e962172dd166f053984806d80b81d06992ec5ce
src/lib/edje/edje_entry.c
src/lib/edje/edje_object.eo
src/lib/edje/edje_private.h
src/lib/edje/edje_util.c