Fix race condition between ibus_bus_set_global_engine() and ibus_bus_get_global_engine().

If focus moves between the two API calls, ibus_bus_get_global_engine() might return an unexpected engine name:

1. context A is focused, and the current global engine is "X".
2. ibus_bus_set_global_engine("Y") is called.
3. a user moves the focus from A to B. First, A's engine is set to NULL in bus_ibus_impl_set_focused_context(). Then, in the same function, B's engine is set to "X" (not "Y") since the _ibus_set_global_engine asynchronous call is not finished yet.
4. ibus_bus_set_global_engine("Y") async call successfully finishes. Context A's (not B's) engine is set to "Y", but context B, which has a focus, is not updated.
5. ibus_bus_get_global_engine() is called.

expected:
Y is returned.

actual:
X is returned. Since the context B has a focus, and B's engine is X.

BUG=http://crosbug.com/11031
TEST=see the bug

Review URL: http://codereview.appspot.com/4063041

Do not allow clients to destroy the server-side config service for safety.

Unset the own flag of priv->config so that the config proxy object never destroys the config service even if a client wrongly unref()s the priv->config object.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/4060041

Fix possible SEGV in ibus_bus_get_global_engine.

The g_variant_get returns NULL when the IPC call times out.

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3974043

Handle GlobalEngineChanged signals from ibus-daemon.

BUG=http://crosbug.com/11011
TEST=manually

Review URL: http://codereview.appspot.com/4037042

On ChromeOS, we don't have to (and don't want to) set default preload engines.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/4054041

Set nonzero serial number for RegistryChanged / EngineChanged signals.

libdbus expects each message serial is greater than 0.
https://code.google.com/p/ibus/issues/detail?id=1186

BUG=none
TEST=manual test described in the issue#1186

Review URL: http://codereview.appspot.com/4050041

Fix typo in the default configuration of the trigger hotkeys.

According to setup/keyboardshortcut.py, "Control" seems to be correct.

BUG=none
TEST=compile ibus with --enable-memconf, start ibus-daemon, start ibus-setup, click "..." button for the "Enable or disable" hotkey, then verify that the "Control" checkbox in the pop-up dialog is checked from the beginning.

Review URL: http://codereview.appspot.com/4017042

Set correct g-flags to proxy objects.

This change is similar to https://github.com/ibus/ibus/commit/19247aac5a348185a0e180f254ab3858c5f85703.

The flag can prevent unnecessary communications between ibus-daemon and e.g. engines like this:

$ . ~/.config/ibus/bus/*-unix-0 ; dbus-monitor --address $IBUS_ADDRESS 2>&1 | tee /tmp/ibus.log
...
method call sender=org.freedesktop.DBus -> dest=(null destination) serial=3 path=/org/freedesktop/IBus/Factory; interface=org.freedesktop.IBus.Factory; member=CreateEngine
   string "mozc-jp"
method return sender=:1.15 -> dest=org.freedesktop.DBus reply_serial=3
   object path "/org/freedesktop/IBus/Engine/1"
method call sender=org.freedesktop.DBus -> dest=(null destination) serial=4 path=/org/freedesktop/IBus/Engine/1; interface=org.freedesktop.DBus.Properties; member=GetAll
   string "org.freedesktop.IBus.Engine"
method return sender=:1.15 -> dest=org.freedesktop.DBus reply_serial=4
   array [
   ]

Calling an engine's org.freedesktop.DBus.Properties.GetAll method when the engine is created is unnecessary for ibus-daemon, while it's not harmful though.

BUG=none
TEST=verified using dbus-monitor that ibus-daemon does not call GetAll anymore.

Review URL: http://codereview.appspot.com/4036041

Fix a gcc warning seen in Chromium OS build.

Please note that gcc's warn_unused_result check is enforced on the build environment. Anyway, I believe it's good to check the return value in this case.

BUG=none
TEST=ran emerge-x86-generic ibus.

Review URL: http://codereview.appspot.com/3904044

Remove a gcc warning.

ibusxml.c: In function 'ibus_xml_parse_file':
ibusxml.c:213: error: 'retval' may be used uninitialized in this function

BUG=none
TEST=ran make CFLAGS="-Wall -O2"

Review URL: http://codereview.appspot.com/3858044

Fix typo in ibus_bus_exit.

BUG=none
TEST=ran make CFLAGS="-Wall -O2"

Review URL: http://codereview.appspot.com/3900045

Fix the return value of ibus_bus_register_component so it returns TRUE on success.

Do the same for ibus_bus_set_global_engine as well. Currently it always returns TRUE regardless of the result of the method call.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3902043

Change the return type of ibus_bus_exit to make it compatible with ibus-1.3 API.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3923042

Set correct sender and interface when sending org.freedesktop.IBus signals.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3775042

Set the "g-default-timeout" property for ibusconfig, ibusinputcontext, and ibusbus objects. The GDBus default timeout (25 seconds) seems to be too long for IBus.

BUG=none
TEST=manually, with IBUS_BUS_TIMEOUT=-3,-2,-1,0,1,1000,6000,abc,1abc,abc1

Review URL: http://codereview.appspot.com/3799049

Use the G_DBUS_CALL_FLAGS_NO_AUTO_START flag just in case.

BUG=none
TEST=manually on Chromium OS.

Review URL: http://codereview.appspot.com/3774043

Do not write empty string to "preload_engines" for safety.

Without the change, bus_ibus_impl_set_default_preload_engines() might erase an existing preload_engines config when --timeout option for ibus-daemon is too short for ibus_config_get() to success.

BUG=none
TEST=manually with 'ibus-daemon --timeout=1'

Review URL: http://codereview.appspot.com/3789046

Call ibus_hotkey_profile_remove_hotkey_by_event in bus_ibus_impl_set_trigger before registering the fallback hotkey, just in case.

Probably it's better to call the remove function in bus_ibus_impl_set_trigger just as bus_ibus_impl_set_hotkey already does, so that a wierd error message like 'IBUS-CRITICAL **: ...: file ibushotkey.c: line 381 (ibus_hotkey_profile_add_hotkey): should not be reached' will not be shown.

The fallback hotkey could be registered twice, via bus_ibus_impl_init and _dbus_name_owner_changed_cb, if the --timeout parameter is too small to talk to the configuration daemon, for example.

BUG=none
TEST=ran ibus-daemon with --timeout=1 and verified that the IBUS-CRITICAL message above was not shown.

Review URL: http://codereview.appspot.com/3796044

Replace g_return_if_fail with g_warning since the error message 'desc != NULL' seems confusing.

BUG=none
TEST=manually. removed all engines using ibus-setup and ran ibus-daemon, then press a trigger hot-key.

Review URL: http://codereview.appspot.com/3786047

Fix --timeout command line option of ibus-daemon.

On 1.3.99, currently the option is just ignored and '-1' (i.e. the default timeout of gdbus which is 25 seconds in glib-2.26.1) is always used. We should fix this since the default, 25 secs, is too long for some platforms like Chromium OS.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3784047

Support hotkeys that use Super/Hyper modifiers.

BUG=1175
TEST=checked that Super+space and Hyper+space work as intended with and without the use_sys_layout option enabled.

Review URL: http://codereview.appspot.com/3856041

Reply an error message to sender, if ibus-daemon can not forward method call message successfully.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3825042

Fix g_variant_get() call in DeleteSurroundingText signal handler.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3820042

Fix API compatibility issue in ibus_config_new.

On ibus-1.3, ibus_config_new returns NULL when ibus-gconf is not started yet, but on 1.4 it returns a valid, non-NULL IBusConfig object. This patch fixes the discrepancy by changing the behavior of ibus_config_new of 1.4.

If we don't return NULL when ibus-gconf does not exist, successive calls e.g. ibus_config_set_value will fail with a cryptic error message like 'IBUS-WARNING **: org.freedesktop.IBus.Config.SetValue: Cannot invoke method; proxy is for a well-known name without an owner and proxy was constructed with the G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START flag'. I believe returning NULL makes it easier to use the ibusconfig.h APIs.

Please note that this patch is particularly important for Chromium OS. Since ibus_config_new is called shortly after ibus-daemon starts on the OS, ibus_config_new is sometimes called before ibus-memconf starts actually. This change helps the glue between chrome and ibus-daemon to remain clean.

BUG=none
TEST=manually on Chromium OS

Review URL: http://codereview.appspot.com/3784043

Create an instance of GDBusProxy with G_DBUS_PROXY_FLAGS_DO_NOT_LOAD_PROPERTIES flag.

Without the flag, on_name_owner_changed() in glib-2.26.x/gio/gdbusproxy.c might invoke a remote method named GetAll in org.freedesktop.DBus.Properties interface, but the method is not implemented in ibus-daemon. Since ibus-daemon ignores the method call, the caller, which is UI thread of Chrome on Chromium OS, will block for 25 seconds (i.e. the default timeout of GDBus) waiting for a reply.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3836042

Fix Alt+Shift hotkey issue for chrome os

BUG=chromium-os:6225
TEST=manual

Review URL: http://codereview.appspot.com/3741043

Fix hot-key handling in inputcontext.c.

Without the fix, some key combinations might not trigger a hot-key event.
For example, one of Chrome OS hot-keys, "press Shift, then press Alt, then release Alt", does not work without the fix.

BUG=none
TEST=manually checked using ChromeOS and IBus-1.4

Review URL: http://codereview.appspot.com/3828041

Add comments to ibusproxy.[ch] and related functions.

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3802042

Fix ibus_input_context_get_input_context() so IBus.Service.Destroy will not be called when the returned object is disposed.

BUG=none
TEST=verified that calling g_object_unref(the_returned_object) does not result in the remote method call.

Review URL: http://codereview.appspot.com/3803041

Don't call nonexistent org.freedesktop.IBus.InputContext.Destroy.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3800041

Fix typo in introspection_xml[] for InputContext.

https://github.com/ibus/ibus/commit/ed778cd7cfc78eb39fc74167f2169b69f4d72a6b was incomplete, sorry.

BUG=none
TEST=verified that properties can be changed via input context on Chrome OS.

Review URL: http://codereview.appspot.com/3795041

Init GError before calling g_dbus_proxy_call*().

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3669041

Fix gtk version check for deprecated keysyms.

From the git log of gtk3, they started adding _KEY from 2.91.0 not 2.90.0.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3670041

Disconnect closed signal from GDBusConnection before creating a new connection.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3668041

Fix assertion failure in ibusbus.c.

The following code does not work since the underlying connection for IBus-1.4 is not a sub class of IBusObject.

        ibus_object_destroy ((IBusObject *)priv->connection);
        g_assert (priv->connection == NULL);

BUG=none
TEST=ran Chrome for Chrome OS.

Review URL: http://codereview.appspot.com/3658041

Fix type mismatch in src/ibuspanelservice.c.

BUG=none
TEST=ran candidate_window for Chrome OS.

Review URL: http://codereview.appspot.com/3661041

Use unsigned for the parameter of the PropertyActivate message.

unsigned would be better since the Python binding and engine proxy use unsigned for this purpose.

BUG=none
TEST=ran Chrome for Chrome OS.

Review URL: http://codereview.appspot.com/3660041

Fix type mismatch in the PropertyActivate message.

src/ibusinputcontext.c sends the massage with "(si)" but the daemon use the massage as "(&su)". This triggers assertion failure on Chrome OS.

BUG=none
TEST=change ibus-mozc's property on Chrome OS via the IME menu.

Review URL: http://codereview.appspot.com/3636041

Fix crash in im-ibus.so

BUG=chromium-os:9868
TEST=manual

Review URL: http://codereview.appspot.com/3401041

Focus out/in fake input context if the input window changed.

BUG=chromium-os:8855
TEST=manual

Review URL: http://codereview.appspot.com/3199042

Focus out the fake context in im-ibus.so, when a real context gets the focus.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3561042

Add fake context back in IBusIMContext.

The fake context is removed by mistake in ibus-1.4. It is necessary for Chrome OS.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3574042

Fix typo and annotation.

Fix typo in IBusEngineDesc:rank doc and GI transfer mode for
ibus_component_get_engines().

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3614041

Fix GI transfer mode annotation in ibus_bus_list_*engines() comment.

See https://bugzilla.gnome.org/show_bug.cgi?id=635248.  Also, do not mark those
functions as "not implemented", since they are apparently implemented.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3274044

Add #include <ibuspanelservice.h> to ibus.h.

Add #include <ibuspanelservice.h> to ibus.h so an external project (e.g. Chromium OS project) could implement its own panel component in C/C++.

Review URL: http://codereview.appspot.com/3411041

Remove unused header file, src/ibusconfigprivate.h

Review URL: http://codereview.appspot.com/3412041

Fix compile errors on Chromium OS.

This is for the master branch of ibus.

BUG=crosbug.com/9685
TEST=ran emerge-x86-generic ibus.

Review URL: http://codereview.appspot.com/3368041

Do not call g_critical when ibus->config is NULL since it's not an error.

The change is originally for chromeos branch of ibus (http://codereview.appspot.com/3304043/) but I believe this is also good for the master branch of ibus.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3269043

Add the "destroy" signal handler back following Peng's suggestion at codereview.appspot.com/3293042/.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3335041

Add function comments to bus/panelproxy.[ch].

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3293042

Fix unexpected client capabilities in ibus-daemon.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3323041

Change the order of engine method calls for consistency.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3304042

Add function comments to bus/inputcontext.[ch] and bus/engineproxy.[ch].

Removed a temporary hack for PREEDIT_CONDITION by adding bus_ibus_impl_is_embed_preedit_text function to ibusimpl.c.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3321041

Re-enable the use_sys_layout feature.

Currently use_sys_layout cannot be disabled.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3306041

Pass cancellable object to bus_factory_proxy_create_engine

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3304041

Clean up the code of focus switch logic, and fix a crash when focused context is destroyed.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3192042

Fix some crashs when engine processes are killed.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3152042

Use g_object_add_weak_pointer to make code simpler.

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3252041

Add generated files into .gitignore

Avoid comparison of unsigned expression

comparison of unsigned expression < 0 is always false

Add assertions to connection_destroy callback

Remove some unused header files.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3218041

Fix NULL of focused input context

Add comments to bus/connection.[ch].

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3164041

Add comments to bus/factoryproxy.[ch].

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3163041

Add comments to bus/registry.[ch].

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3162041

Add comments about thread safety.

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3156041

Add comments to bus/component.[ch].

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3161041

Add comments to bus/ibusimpl.[ch].

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3154041

Fix no defined '_' in EngineAbout

Add introspection data for org.freedesktop.IBus.Panel and remove Destroy method.

Add introspection data for org.freedesktop.IBus.Panel interface.
And remove Destroy method from Panel interface, because it is moved to
org.freedesktop.IBus.Service interface.

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3137041

Add comments to bus/dbusimpl.[ch].

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3132041

Make sure send the reply of method IBus.Exit out before the daemon exits.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3133041

Add global variable DOMAINNAME to replace "ibus10" in everywhere.

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3093042

Fix ibus_version in configure and translation domain in ibus-setup.

Always read Window as 32 bits integer to fix problem in ppc64.

Use "org.freedesktop.IBus" instead of "org.freedesktop.DBus" for the well-known bus name for the input context proxy.

Though "org.freedesktop.DBus" works fine, since bus_dbus_impl_connection_filter_cb in bus/dbusimpl.c treats the two bus names equally, "org.freedesktop.IBus" which matches the interface name looks more natural.

BUG=none
TEST=manually done

Review URL: http://codereview.appspot.com/3128041

Remove block call in ibus-daemon, and use async call instead and clean up code.

1. Do not use poll factory of a component. Sometime, it will cause a dead lock of ibus: ibus-daemon are waiting reply from engine, and engine are also waiting for reply from ibus-daemon.
2. Move some API from IBusComponent to BusComponent, Because of those API is for internal using only
3. Add a fake input context in server side to support switching input method when no context has focus.
4. Remove fake input context in imcontext, because we added the server side fake context

BUG=none
TEST=manual

Review URL: http://codereview.appspot.com/3077042

Add function comments to bus/server.[ch]

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3040041

Add function comments to bus/main.c.

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3036041

Delete src/ibusserver.[ch] that are no longer used.

BUG=none
TEST=none

Review URL: http://codereview.appspot.com/3018042

Disable snooper in chromium, chrome, firefox and gnome-do

Those applications has problems with snooper enabled. So update the
NO_SNOOPER_APPS to disable snooper for those applications by default.

BUG=ibus:1140
TEST=manual

Remove a debug output

Remove GDK_DISPLAY, this macro has been removed in gtk3

http://live.gnome.org/GnomeGoals/RemoveDeprecatedSymbols/GTK%2B

Update python test scripts for gobject-introspection

Remove the ability to attach to a status icon with libnotify 0.7.0

Refine some code and remove some dead code.

Fix some issues within codereview.

Fix crash when icon of property is Null

Fix UnsetValue name.

Fix serialized order.

Rename ibus_engine_desc_new2 and ibus_component_new2 to *_new_varargs.

wip.

Fix a typo.

Fix a typo in client/gtk3/Makefile.am

Fix some problems in debian packages.

Add i18n.py.

Fix some merge failed.

Rewrite memconf in c languages.

Emit NameOwnerChanged signal with non-zero serial to make libdbus happy.

Add metadata file to fix some object ownership correctly