Branch: refs/heads/1.3.y
Home:   http://github.com/ibus/ibus

Commit: 2100c1f781fec3a2157e57514d413a977042d84c
    http://github.com/ibus/ibus/commit/2100c1f781fec3a2157e57514d413a977042d84c
Author: Yusuke Sato <[email protected]>
Date:   2010-10-19 (Tue, 19 Oct 2010)

Changed paths:
  M bus/ibusimpl.c

Log Message:
-----------
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


Commit: 4be55968e7f98d1b042068b47be82395715bf9bc
    http://github.com/ibus/ibus/commit/4be55968e7f98d1b042068b47be82395715bf9bc
Author: Peng Huang <[email protected]>
Date:   2010-10-20 (Wed, 20 Oct 2010)

Changed paths:
  M bus/ibusimpl.c

Log Message:
-----------
Always consume the hotkey, even if the hotkey associated engine already 
activated.

BUG=6376
TEST=manual

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


Commit: 9ae13a3d95c7dc775084c58339e42375622df83e
    http://github.com/ibus/ibus/commit/9ae13a3d95c7dc775084c58339e42375622df83e
Author: Yusuke Sato <[email protected]>
Date:   2010-10-20 (Wed, 20 Oct 2010)

Changed paths:
  M client/gtk2/ibusimcontext.c
  M configure.ac

Log Message:
-----------
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:

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

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

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

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

yusu...@harapeko:~$ gedit  # no-snoop
yusu...@harapeko:~$ IBUS_DISABLE_SNOOPER="false" gedit  # snoop
yusu...@harapeko:~$ IBUS_DISABLE_SNOOPER="true" gedit  # no-snoop
yusu...@harapeko:~$ IBUS_DISABLE_SNOOPER="unknownstring" gedit  # no-snoop
yusu...@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


Commit: 1c2b65dd3fde6891eff44b7b3bf3371897e3a582
    http://github.com/ibus/ibus/commit/1c2b65dd3fde6891eff44b7b3bf3371897e3a582
Author: Yusuke Sato <[email protected]>
Date:   2010-10-20 (Wed, 20 Oct 2010)

Changed paths:
  M bus/ibusimpl.c

Log Message:
-----------
Remove compiler warnings.

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

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


Commit: 8c131cfee02c9104495f11456eb40dc162458a2f
    http://github.com/ibus/ibus/commit/8c131cfee02c9104495f11456eb40dc162458a2f
Author: Yusuke Sato <[email protected]>
Date:   2010-10-20 (Wed, 20 Oct 2010)

Changed paths:
  M ui/gtk/candidatepanel.py
  M ui/gtk/panel.py

Log Message:
-----------
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


Commit: 96b64a2eff1fb4c9bec4e712b47b23a7900f44e7
    http://github.com/ibus/ibus/commit/96b64a2eff1fb4c9bec4e712b47b23a7900f44e7
Author: Peng Huang <[email protected]>
Date:   2010-10-21 (Thu, 21 Oct 2010)

Changed paths:
  M client/gtk2/ibusimcontext.c
  M src/ibusbus.c

Log Message:
-----------
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


Commit: 15db48fc217cb6e91c91e1819f2347dfdac905e0
    http://github.com/ibus/ibus/commit/15db48fc217cb6e91c91e1819f2347dfdac905e0
Author: fujiwarat <[email protected]>
Date:   2010-10-21 (Thu, 21 Oct 2010)

Changed paths:
  M data/ibus.schemas.in
  M setup/keyboardshortcut.py

Log Message:
-----------
Fix keyval and state in __keycode_button_clicked_cb


Commit: acf2a46cdce536945d56ecc2a581fda38470dfe5
    http://github.com/ibus/ibus/commit/acf2a46cdce536945d56ecc2a581fda38470dfe5
Author: fujiwarat <[email protected]>
Date:   2010-10-21 (Thu, 21 Oct 2010)

Changed paths:
  M client/gtk2/ibusimcontext.c
  M ui/gtk/panel.py

Log Message:
-----------
Fix CPU 100% usage with signal.SIGCHLD


Commit: 74cf9101c7d1700d6add470df9db59fe3593f3a1
    http://github.com/ibus/ibus/commit/74cf9101c7d1700d6add470df9db59fe3593f3a1
Author: Peng Huang <[email protected]>
Date:   2010-10-21 (Thu, 21 Oct 2010)

Changed paths:
  M ui/gtk/panel.py

Log Message:
-----------
Catch exceptions from os.kill.

BUG=none
TEST=manual

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


Commit: 2b1fa3dc2a2cb57484de5506f4081f933ba1c0a4
    http://github.com/ibus/ibus/commit/2b1fa3dc2a2cb57484de5506f4081f933ba1c0a4
Author: Peng Huang <[email protected]>
Date:   2010-10-21 (Thu, 21 Oct 2010)

Changed paths:
  M bus/Makefile.am
  M bus/test-stress.c

Log Message:
-----------
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


Commit: 4ed0cee9a76e1123fc116ba4a251c8cfba407990
    http://github.com/ibus/ibus/commit/4ed0cee9a76e1123fc116ba4a251c8cfba407990
Author: Peng Huang <[email protected]>
Date:   2010-10-21 (Thu, 21 Oct 2010)

Changed paths:
  M src/test-keymap.c

Log Message:
-----------
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


Commit: 5f5017accfb859cac8829ef9d8b19547b5cc1c41
    http://github.com/ibus/ibus/commit/5f5017accfb859cac8829ef9d8b19547b5cc1c41
Author: Peng Huang <[email protected]>
Date:   2010-10-21 (Thu, 21 Oct 2010)

Changed paths:
  M configure.ac

Log Message:
-----------
Bump the version to 1.3.8


-- 
You received this message because you are subscribed to the Google
Groups "ibus-devel" group.
iBus project web page: http://code.google.com/p/ibus/
iBus dev group: http://groups.google.com/group/ibus-devel?hl=en

回复