Revert "wl-drm: Set initial compositor pointer values base on ecore_evas_pointer_position"

This reverts commit 16e9126302c3a95f84921b62dbe547b1c112cd2f.

Reverting this due to crash

wl-drm: Set initial compositor pointer values base on ecore_evas_pointer_position

This addresses the issue of initial compositor pointer values not
being proper. They are 0,0 on startup, however things like ecore_drm
will center the mouse on startup. The e_comp pointer values should
reflect this.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

module/geolocation: Stop setting gadget icon from start and stop callbacks

We keep track of this now by monitoring the InUse property which works also
if other clients use the service.

modules/geolocation: Follow InUse property change to give UI indication

The InUse property of the manager object is available in newer GeoClue2
versions. It shows if the service is in use by any application at the
moment. We use this to indicate the usage in the UI.
A blue geolocation icon means the service is in use, the grey icon means
it is inactive.

modules/geolocation: Make module work with newer GeoClue2

In newer GeoClue2 version, tested with 2.1.10, we need to set
the DesktopId and AccurancyLevel to get a location object.

modules/geolocation: Only display a valid altitude value

If our backends does no know the altitude it sets it to -DBL_MAX as 0
is a valid altitude value. Check for it and  display N/A if it is not valid.

e screenlock config diloag - note insecureness for personal pw/pin

these store pin/pw in your user config files - it may be primitively
hashed to obscure it, but it's there. it never pretended to have
secure storage and even saved cleartext until e19. make sure people
are aware

po: Remove unused e_int_config_randr.c file from POTFILES.in to pass distcheck

This file is still around while not really used. It is also not distributed and
thus fails if po tries to update during distcheck with this file still in list.

unadjust frame coords when calculating wl surface placement during commit

ec->y -> unadjust -> ec->client.y

use pointer hash for wl pixmaps

pass surface resource when creating wl surface pixmaps

Revert "build: Remove generated eldbus files during normal clean and not only maintainer-clean"

This reverts commit 95d701207783bcf58034797468024c05fa59ca03.

do want in tree

add explicit includes for elm color class header in colorclass cfd

disable client color editor for non-X clients for now

wl client pixmaps now use a different method for creating the pixmap id

add wl client event callbacks at lower priority

this ensures that bindings will activate before events are passed to clients

use client geom for comp object centering functions if ec->new_client is set

e-comp-wl: Don't crash if we fail to create a new e_client

This "should" fix T2344 crashing, however I suppose a better question
is ... why is e_client_new returning NULL when it Does find the client
already in the hash...

Signed-off-by: Chris Michael <cp.michael@samsung.com>

e_auth: Add suid helper for lokker own-pw checking on FreeBSD

Summary:
PAM on FreeBSD, unlike on Linux, does not allow users to check their own
password. Instead, we need a suid helper to do it for us. Add such a
helper on FreeBSD.

For now, it is limited to checking users in the local password database
(traditional Unix passwd file). This could and should be extended to use
PAM in a later patch.

Test Plan:
Tested empty pw, wrong pw, correct pw at lock screen; observed correct behavior
in each instance.

Reviewers: q66, zmike

Reviewed By: q66, zmike

Subscribers: cedric, seoz

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

Auth: try harder when clearing passwords.

Based on http://www.daemonology.net/blog/2014-09-04-how-to-zero-a-buffer.html

Thanks again to Matthew Garrett, for pointing this out on twitter.

Module teamwork: Stop using a deprecated function.

E auth: improve clearing out passwords from memory.

Optimising compilers (like gcc/clang with -O1 or above) were optimising
out the memset(). Until link time optimisations are good enough, this
will prevent them from doing so. The best solution would be to use
memset_s() (c11), though it's not readily available yet. This is the
first step towards using memset_s() with a fallback for systems who
don't have it. A better solution, is to put it in Eina, to prevent LTO
completely. This will have to be done after the EFL release.
Even this is not entirely safe though, but at least it protects us from
some memory disclosure issues.

This doesn't solve the fact that we may store a copy of the password in
other places, like the input system. We need to address that too.

Thanks to Matthew Garrett for pointing this out or Twitter.

e - e_client - fix added printf

665caad451c7184bb85b85067c491d15d3dfc1ea fix added a printf i didn't
notice - fix.

e client - desk wait - ref it properly and clean up delfn

delfn's on desks being referred to as profile.wait_desk were never
cleared when client was freed. als the desk itself never got an obj
reference - fix that and ref/unref it.

e - xsettings - calloc buffer since we leave 3 of 4 bytes unset

the first 4 bytes is a msb/lsb thing for xsetttings - we ignore 3 of
the 4 bytes. thus we should just calloc the whole buffer to ensure its 0

Fix wayland compositor on 32 bit systems

Summary:
_e_elm_win_trap_show() treats Ecore_Window as a 64bit data type when
it's actually the same size as the system's pointer type.

 #24HourFullPowerNoToiletNoShower

Reviewers: devilhorns, zmike

Subscribers: cedric

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

fix T2194

fix submenu visibility during scroll

sometimes while quickly moving the cursor through menu items with submenus
the visibility flag would toggle more quickly than the menu object's visibility.
this is no longer the case

attempt to decode notification image uris with efreet before showing them

fix T1939

e_backlight_main: fix compiling when not having Eeze

Summary:
EINA_UNUSED is defined in Eina. Since we are not using the program at all,
let's void main()'s arguments instead of trying to ignore them.

Test Plan: tested only on OSX, unbreaks the build.

Reviewers: zmike

Subscribers: cedric

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

clear app menu cache any time a desktop is edited internally

ref T2271

add function for clearing app menu cache

ref T2271

swap key -> keyname when attempting to bind lparen/rparen keys

this is impossible to grab on X, so ensure the generated binding is usable

fix T2059

fix clock popup widget parenting

use client geometry for starting unmaximize calculations

the resize operations in this function operate off client geometry, so
stop using frame geometry

fix T2170

wl-drm: Use ecore_evas_new with an engine name rather than direct ecore_evas_drm_new calls

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Make some randr messages translatable

Wizard: Update icon theme search list

Summary:
@fix The list of icon themes in the search list is quite out of date
it doesn't include the icon themes used by gnome3 or kde4/5 this means
for a lot of users at first boot e has alot of missing icons.

The Icon sets that have been added are as follows
      "Oxygen",            /* KDE 4 */
      "Adwaita",           /* Gnome 3 */
      "Breeze",            /* KDE 5 */
      "HighContrast"

This change will cause T1732 and T1923 to not occur for almost all users
but it does not fix them properly the code should be modified to fall back
to pick up icons from fallback locations such as the hicolor theme if no
icon theme is set.

It would be nice if someone could backport this to e19 I don't know how to

Reviewers: bu5hm4n, zmike, raster

Reviewed By: raster

Subscribers: cedric

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

PackageKit: Fix a crash properly

This reverts and tries to actually fix:

commit b30892056e0a527545c8a022cd2b3cdef0ec85af
Author: Jean-Philippe Andre <jp.andre@samsung.com>
Date:   Fri Apr 17 10:30:06 2015 +0900

    PackageKit: Fix crash on error

    Sometimes a "normal" error occurs and then the callback is
    called with no context data. Then crash boom sigsegv happens.

remove support for wayland builds on 32bit

this doesn't work for some reason and there are no plans to fix it anytime soon.

ref some infinite number of tickets

PackageKit: Fix crash on error

Sometimes a "normal" error occurs and then the callback is
called with no context data. Then crash boom sigsegv happens.

fix unoptimized stringshare usage in keybindings cfd

prune no_display desktops from evry apps plugin

probably fix T2256

fix evry app listing list leak

fix pixmap image clearing based on pixmap type

!@%!@$#!# borkerhorns!!!!!!

invalidates T2291

e_client: Remove self assignment from e_comp

Hopefully the last one of these self assignments.

CID 1291840

systray: Remove self assignment from e_comp

Another left over from the e_comp refactor.

CID: 1291839

gadman: Remove self assignment of e_comp

This looks like a left over from the e_comp refactor.

CID: 1291838

mailmap: update q66

shot - only access e client if not null

fix evry comparison macro to work as expected

missing parens = !operating as intended in many cases

fix return value in backlight helper exe on non-freebsd systems

ref D2337

fix nocomp_override conditional which would inaccurately toggle nocomp

hooray gcc 5.0

L10N: Update Finnish translations

tempget: Add lm75(4) temperature source for non-x86 FreeBSD

Summary:
Another suggestion from Dmitry Luhtionov. I am told this sensor (on the
i2c bus) is more likely to be present on non-x86 systems (ARM, etc).

Reviewers: zmike, q66

Reviewed By: q66

Subscribers: cedric, seoz

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

backlight: Use basic ACPI sysctl knob on FreeBSD (kernel)

Summary:
Unconditionally build the suid-helper _backlight program; conditionalize
behavior on Eeze, FreeBSD (doing nothing on neither). Add logic to set
the FreeBSD sysctl in a similar manner to udev devices on Linux.

Add _bl_sys_find/_get helpers for FreeBSD that check for and consult the
video ACPI sysctl.

Test Plan:
Seems to work okay on my laptop (with EINA_CPU_FAKE=1 to workaround a threadq
race that I believe is unrelated -- T2287).

Reviewers: zmike, q66

Reviewed By: q66

Subscribers: cedric, seoz

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

Generate *all* music control files in src tree.

This fixes once and for all out of tree builds.

allow client icon change events in ibar client prop handler

fix T2284

set E_Exec_Instance desktop during client idler if none present

ref T2284

check correct number of params passed to internal/desk cfd

fix T2221

only set shot window layer if taking window shot

fix T2282

fix desktop editor entry widget parenting

fix T2283

do not attempt to re-center fullscreen/maximized internal windows

fix T2279

modules/bluez4: fix duplicated symbol

Summary:
Global variable should be declared as extern to avoid symbol duplication.
Fix compiling of ebluez4 module on OS X.

Test Plan:
Since Enlightenment cannot (yet) be fully compiled on OS X, I cannot test
the real-life impact of this change.

Reviewers: zmike, raster

Subscribers: cedric

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

toggle comp_hidden and grab input while client color editor is active

Tiling: don't tile centered windows.

This broke with the change to elm (so within this release cycle). Tiling
wasn't properly detecting windows who become centered after creation as
there was no e event for that. This is now fixed, and a blanket solution
for similar gotchas has been implemented (otherwise known as a hack).

Thanks to Mike for helping out.

tempget: Add more temperature sources for FreeBSD

Summary:
Not all motherboards have 'hw.acpi.thermal.tz0.temperature' sensor.
Fallback to 'dev.cpu.0.temperature' (provided by coretemp(4) or
amdtemp(4)), or 'dev.aibs.0.temp.0' (aibs(4)) (per suggestion of Dmitry
Luhtionov).

Test Plan: Ran `src/modules/temperature/tempget` manually with some debugging printfs and various sensors commented out. I can only test tz0 and dev.cpu (coretemp(4)) on my system.

Reviewers: zmike, q66

Reviewed By: q66

Subscribers: seoz

Projects: #enlightenment-git

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

e-comp-wl: Remove unused variable

Signed-off-by: Chris Michael <cp.michael@samsung.com>

MEDIUM: modules/shot: switch to https://

This is needed since www.enlightenment.org is now using https:// only due to
login forms

build: Remove generated eldbus files during normal clean and not only maintainer-clean

These generated files should be cleaned up during a normal run of make clean.

make connman agent dialog resizable and autosized

fix T2279

Module Music control: Don't build if disabled.

The whole point of supporting disabling modules is to actually
be able to disable them. :).

@fix.

add xkb options scroller as subobj to toolbook widget

ensure chain of e_widgets for cfd changed policy

fix T2146

modules/battery: fix compiling on OS X

Summary:
Battery module relies on Apple's CoreFoundation (CF), but CF's CFLAGS
were not provided to this module.

Test Plan: Enlightenment now compiles on OS X.

Reviewers: zmike, raster

Subscribers: cedric

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

remove cslider widget

no longer used

remove csel widget

no longer used

redo color dialog to use elm colorselector

this is waaaaaaay less functional and good-looking, but at least it doesn't
crash

make entry widget call changed callbacks on "changed" again

this caused infinite loops before, but now I'm removing the things that looped.

fixes a lot of config dialogs

ref T2271

don't redo min size hints on e_dialog show

these are guaranteed to be wrong because elm won't provide accurate size
hints without some arm twisting

force dialogs to recalc content sizes before setting min size

fix entry sizing in file props dialog

ref T2271

set xkb dialog as changed after adding a new layout

ref T2271

set desktop editor appselector dialog to be resizable

ref T2271

match client dialogs to client's layer

a small annoyance for literally over a decade has been that window
dialogs would sometimes start behind their owners. now they don't.

raise deskmirror windows on startup

these get added bottom -> top on creation, so ensure that they are raised

always lower e_layout children upon packing

fixes issues where a new child added would appear above other child objects
if the new child was meant to be stacked below other children

mostly noticeable in deskmirror

remove layer setting from deskmirror

smart child objects can't have layers. apparently.

Mixer Pulse: Fix trivial use-after-free.

Summary: Don't use memory after freeing it.

Fixes T2274

@fix

Reviewers: raster, zmike

Subscribers: abyomi0, zmike

Projects: #enlightenment-git

Maniphest Tasks: T2274

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

Commit message updates by: Tom Hacohen

add options to randr2 to ignore acpi and hotplug randr events.

don't crash on wl shutdown

devilhorns? more like crasherhorns imo

e_comp: Only need 'type' variable if we are building with HAVE_WAYLAND

Signed-off-by: Chris Michael <cp.michael@samsung.com>

build: Use EFL_WITH_BIN instead of E homegrown one.

We already have the efl_binary.m4 macro available with EFL_WITH_BIN which
basically is the successor of this macro.

build: Allow to define path for eldbus-codegen binary

Since the geolocation and music-control changes we need eldbus-codegen to
generate some files. Not all setups have this in their normal $PATH so let
allow these setups to set the correct path during configure. This allows our
Jenkins setup as well as others to work again.

remove all debug output from mixer module

fix T2274

only call e_comp_wl_shutdown() for wayland compositors

prepend geolocation build dir to cppflags

elocation headers conflict with these, so ensure that our headers get picked up first

 #glimaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

music-control: Mimic explicit file deps fix from geolocation.

This was already firxed for geolocation in
7a1163a1adb942f49380e848c809129c2c0b9c89 but I forgot to integrate it in
my last patch. Thanks to Scimmia for pointing it out.

everything: Actual return something in non wayland only-mode

While making it work for wayland only the return got missing.

everything: win variable is only used inside ifdef.

We only use this in not wayland only mode so ifdef the variable as well.

music-control: Generate eldbus files during compile

Better have up to date eldbus files generated from the xml instead of
keeping the generated files around.

 Bugfix: e dialog: don’t crash when using the escape key to close.

 Commit 99589eb5af346add9fe912a768ed006416cb2220 introduced this bug as the callback creation wasn’t changed to suit the new prototype and stil passed a window object.