atspi: move atspi bridge init to elm_run 75/257375/2
authorShinwoo Kim <cinoo.kim@samsung.com>
Fri, 23 Apr 2021 06:59:13 +0000 (15:59 +0900)
committerLukasz Oleksak <l.oleksak@samsung.com>
Tue, 11 May 2021 11:43:34 +0000 (11:43 +0000)
commitf26c4fef8f7f6b8944071279993181dbb00f57ca
tree7b89cd0e14eea536a42ff27d645e5894be8ff0b5
parentc3b7b388f3f1bc0bc33cbbb859c85485ea767fcd
atspi: move atspi bridge init to elm_run

We moved initialization to elm_run by following commit already.

   commit 273ad815147a7916c426b6dd60fe45a523ccf048
   Author: Lukasz Stanislawski <l.stanislaws@samsung.com>
   Date:   Mon Nov 27 14:18:57 2017 +0530

       atspi: move atspi initialization to elm_run.

       Change-Id: Ib0656d1312367ee214f4cb2b236f28a4d73a1dfe

But in accident, it resurrected with following commit.

   commit 87d7ed882cc130dd92c1bd7677e3223fc0fe6870
   Author: Alastair Poole <netstar@gmail.com>
   Date:   Mon Jul 30 11:42:28 2018 -0400

       efl_selection: reintroduce missing ELM_CNP_EVENT_SELECTION_CHANGED symbol.

       Summary:
       During the 1.21 development cycle the symbol ELM_CNP_EVENT_SELECTION_CHANGED was lost.

       The commit which introduced this:  e88bbaa1e314751e5c1a04dec7f1f179188428ae

       This patch re-introduced the symbol and expected behaviour.

       Reviewers: #committers, zmike, bu5hm4n, herb

       Reviewed By: #committers, zmike

       Subscribers: herb, ManMower, segfaultxavi, stefan_schmidt, cedric, zmike

        Tags: #efl

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

This could be migration error.
So I would like to move atspi bridge init to elm_run again.
AFAIK, if we create DBus connection at elm_init, there is a security issue.
I am not sure if this is still living issue or not.
Because we have done atspi init from 2018 (above second commit).

Change-Id: I0ca58fa915634cf75367383a3f4c38eb8657d0f1
src/lib/elementary/elm_main.c