Page MenuHomePhabricator

Efl.Ui.Progressbar_Legacy: Prevent unnecessary signal emit for contents
ClosedPublic

Authored by jsuya on May 11 2020, 10:13 PM.

Details

Summary

When call elm_progressbar_add, theme_apply of layout is called and
_efl_ui_progressbar_legacy_efl_ui_widget_theme_apply is called.
Then it calls the signal emit for the icon.
this call is unnecessary.

Test Plan

N/A

Diff Detail

Repository
rEFL core/efl
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
jsuya created this revision.May 11 2020, 10:13 PM
jsuya requested review of this revision.May 11 2020, 10:13 PM
jsuya updated this revision to Diff 30212.May 11 2020, 10:42 PM

update code

jsuya updated this revision to Diff 30219.May 12 2020, 5:19 AM

update code - use efl_finalized_get

jsuya updated this revision to Diff 30221.May 12 2020, 5:45 AM
jsuya edited the test plan for this revision. (Show Details)

I removed speed comparison. There are some improvements, but almost the same speed.

jsuya updated this revision to Diff 30222.May 12 2020, 5:46 AM

rebase patch

zmike accepted this revision.May 12 2020, 8:19 AM
This revision is now accepted and ready to land.May 12 2020, 8:19 AM
This revision was automatically updated to reflect the committed changes.