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

Add vala test for config module

Fix vala and gir build issues.

Remove ibusdbus.h

wip.

wip.

Fix some runtime errors.

Change version to 1.3.99

Use va_list in IBusEngineDesc and IBusComponent for back compatibility.

Provided by fujiwarat <takao.fujiwara1@gmail.com>

Conflicts:

bus/engineproxy.c
bus/factoryproxy.c
bus/ibusimpl.c
src/ibuscomponent.c
src/ibusenginedesc.c
src/test-bus.c
src/tests/ibus-global-engine.c

Use gdbus in glib, and get rid of libdbus

Replace s/gdk_drawable_get_size/gdk_window_get_height/ for GTK3.

Use block mode of g_main_context_iteration to avoid consuming too much cpu time.

BUG=none
TEST=manual

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

Use the automake 1.10 in autogen.sh

BUG=ibus:1039
TEST=manual

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

sort combo box by locale strcoll

Fix s/python/@PYTHON@/ in libexec scripts.

The patch is provided by federico.schwindt@google.com

Fix ENV value for NetBSD

Bump the version to 1.3.8

Comment out test-keymap test case.

Comment out test-keymap test case, because it is not automatic test
case. it will cause make distcheck failed.

BUG=none
TEST=manual

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

Limit typing rate to 800 hits/minutes, and fix make distcheck error.

The old version test-stress will send key events to ibus-daemon as fast as possible. It will take all CPU resources, and the daemon, UI and input methods will not get enouch CPU resources to process those events. So all key events will be putted in the buffer. It will causes out of system memory. So I add a limitation of typing rate as 800 hits/minute.

BUG=none
TEST=manual

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

Catch exceptions from os.kill.

BUG=none
TEST=manual

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

Fix CPU 100% usage with signal.SIGCHLD

Fix keyval and state in __keycode_button_clicked_cb

Destroy existing connection before creating a new connection, and only create the fake context once.

BUG=chromium-os:7998
TEST=manual

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

Move the candidate window just above the cursor when the window and a preedit string overlap.

BUG=http://code.google.com/p/ibus/issues/detail?id=1106
TEST=manually done.

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

Remove compiler warnings.

BUG=none
TEST=checked by ./autogen.sh && make

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

Enable key snooper by default again, except Chrome/Chromium browsers, to fix application compatibility issues like 1068.

This change is logically a revert of http://github.com/ibus/ibus/commit/7e715146794d5fa5324885f8d1dcebb8805bc31b

The new behavior is:

1) If IBUS_DISABLE_SNOOPER environment variable is set, and the value of the variable is "" (an empty string) or "0" or "false" or "False" or "FALSE", key snooper is enabled.
2) If IBUS_DISABLE_SNOOPER environment variable is set, and the value of the variable is other than the 5 above, e.g. "1", "true", .., key snooper is disabled.
3) If IBUS_DISABLE_SNOOPER environment variable is not set, and ibus-daemon is explicitly configured with --disable-key-snooper, key snooper is disabled.
4) If IBUS_DISABLE_SNOOPER environment variable is not set, and ibus-daemon is not configured with --disable-key-snooper, the GTK_IM_MODULE, im-ibus.so, checks IBUS_NO_SNOOPER_APPS environment variable:
  4-a) if IBUS_NO_SNOOPER_APPS environment variable is not set, and the application name matches ".*chrome", key snooper is disabled.
  4-b) if IBUS_NO_SNOOPER_APPS environment variable, which should be comma-separated regexps, is set, and one of the regexps matches the application name, key snooper is disabled.
  4-c) otherwise, key snooper is enabled.

Please note that when no configure options nor no environment variables are set, key snooper is enabled on all applications except Chrome/Chromium web browsers. For example, key snooper would be enabled on xchat and gedit by default. I believe the new default behavior would satisfy both Linux desktop and Chromium OS requirements.

Test:

- With ibus built without --disable-key-snooper:

yusukes@harapeko:~$ gedit  # snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER= gedit  # snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER=0 gedit  # snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="0" gedit  # snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="false" gedit  # snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="False" gedit  # snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="FALSE" gedit  # snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="1" gedit  # no-snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="true" gedit  # no-snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="unknownstring" gedit  # no-snoop

yusukes@harapeko:~$ IBUS_NO_SNOOPER_APPS=gedit gedit  # no-snoop
yusukes@harapeko:~$ IBUS_NO_SNOOPER_APPS='g.*dit' gedit  # no-snoop
yusukes@harapeko:~$ IBUS_NO_SNOOPER_APPS='foobar,g.*dit' gedit  # no-snoop
yusukes@harapeko:~$ IBUS_NO_SNOOPER_APPS=foobar gedit  # snoop

yusukes@harapeko:~$ google-chrome  # no-snoop
yusukes@harapeko:~$ IBUS_NO_SNOOPER_APPS=foobar google-chrome  # snoop

- With ibus built with --disable-key-snooper:

yusukes@harapeko:~$ gedit  # no-snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="false" gedit  # snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="true" gedit  # no-snoop
yusukes@harapeko:~$ IBUS_DISABLE_SNOOPER="unknownstring" gedit  # no-snoop
yusukes@harapeko:~$ IBUS_NO_SNOOPER_APPS=foobar gedit   # no-snoop, because IBUS_NO_SNOOPER_APPS is ignored when --disable-key-snooper is specified.

BUG=http://code.google.com/p/ibus/issues/detail?id=1068
TEST=manually done. see above.

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

Always consume the hotkey, even if the hotkey associated engine already activated.

BUG=6376
TEST=manual

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

Fix race condition in bus_ibus_impl_create_engine()

If the bus_ibus_impl_create_engine() function is called right after an ibus_component_start() call, the function might fail getting a factory object.

To avoid the problem, we should use the busy-wait logic even when ibus_component_is_running() returns true.

BUG=http://crosbug.com/7244
TEST=see the bug (comment #4,7,9)

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

Fix a build error of IBus-1.0.gir with gobject-introspection 0.9.6

Fix a typo.

Remove unused dbus-1 package in gir includes

BUG=none
TEST=manual

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

Implement stress tool for ibus.

A test-stress.c sends key message each time by client.
Client.h and client.c store modifier key and send key event.
They check ibus-daemon and engine is alive.

Review URL: http://codereview.appspot.com/2204051
Patch from Hirotake Kobayashi <hkobayash@google.com>.

Fix a build error with GTK3 gdkkeysyms.h

Update gu.po from Sweta Kothari <swkothar@redhat.com>

Fix g_assertion failure in bus_ibus_impl_{load,save}_global_engine_name functions.

Add NULL checks to the functions so they don't abort with assertion failure even if the functions are called before the ibus configuration daemon gets ready.
See http://code.google.com/p/chromium-os/issues/detail?id=6689#c27 for details.

Add some comments about registry

BUG=none
TEST=built fine

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

Add Emacs modeline for .c and .h files.

/* -*- mode: C; c-basic-offset: 4; indent-tabs-mode: nil; -*- */

TEST=confirmed that the modelines for vim and emacs both worked.
BUG=none

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

Move the log handler code from bus/main.cc to src/ibusshare.cc.

The log handler is useful for IBus client programs. For instance,
the log handler was used inside the chromium browser for debugging
IBus startup issue:

http://code.google.com/p/chromium-os/issues/detail?id=6375#c29

(chrome:11824): IBUS-DEBUG: 00:00:46.929430: The socket file is now created (monitoring done).

BUG=none
TEST=manually ran ibus-daemon with and without -v

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

Add codereview.settings.

This is for us to use codereview.appspot.com for code reviews.

Instructions to use codereview.appspot.com:

1. Get depot_tools and add PATH to the directory
http://dev.chromium.org/developers/how-tos/depottools

2. In your local IBus git repository,

% git checkout master
% git checkout -b somefix
make changes
% git cl upload
do code reviews at codereview.appspot.com
once it's done, run "git cl push" if you can commit.
otherwise, ask committers to commit on behalf.

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

Add time info in log message from satorux@chromium.org