Keyboard leaks memory when switching layouts

Bug #1420891 reported by Michael Sheldon
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Critical
Bill Filler
ubuntu-keyboard
Fix Released
Critical
Michael Sheldon
ubuntu-keyboard (Ubuntu)
Fix Released
Undecided
Unassigned
ubuntu-keyboard (Ubuntu RTM)
Fix Released
Undecided
Unassigned

Bug Description

The external libraries used in the word engines are never cleaned up when closed by the QPluginLoader, this results in the keyboard using an additional 2-3mb of RAM every time the layout is switched.

Steps to reproduce

1. Enable multiple layouts

2. Switch between them repeatedly

Expected result

Memory usage should level off to a constant amount

Actual result

Memory usage continues increasing with every layout switch

Tags: ww09

Related branches

Changed in ubuntu-keyboard:
status: New → Confirmed
status: Confirmed → In Progress
assignee: nobody → Michael Sheldon (michael-sheldon)
importance: Undecided → Critical
Changed in canonical-devices-system-image:
importance: Undecided → Critical
milestone: none → ww09-2015
status: New → In Progress
Changed in canonical-devices-system-image:
assignee: nobody → Bill Filler (bfiller)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-keyboard - 0.99.trunk.phablet2+15.04.20150213-0ubuntu1

---------------
ubuntu-keyboard (0.99.trunk.phablet2+15.04.20150213-0ubuntu1) vivid; urgency=medium

  [ CI Train Bot ]
  * Resync trunk
  * New rebuild forced.

  [ Michael Sheldon ]
  * Fix memory leak when unloading word engines (LP: #1420891)
 -- CI Train Bot <email address hidden> Fri, 13 Feb 2015 04:09:03 +0000

Changed in ubuntu-keyboard (Ubuntu):
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-keyboard - 0.99.trunk.phablet2+15.04.20150213.2~rtm-0ubuntu1

---------------
ubuntu-keyboard (0.99.trunk.phablet2+15.04.20150213.2~rtm-0ubuntu1) 14.09; urgency=medium

  [ CI Train Bot ]
  * Resync trunk
  * New rebuild forced.

  [ Michael Sheldon ]
  * Add a brief delay before evaluating swipes (LP: #1421742)

ubuntu-keyboard (0.99.trunk.phablet2+15.04.20150213-0ubuntu1) vivid; urgency=medium

  [ CI Train Bot ]
  * Resync trunk
  * New rebuild forced.

  [ Michael Sheldon ]
  * Fix memory leak when unloading word engines (LP: #1420891)

ubuntu-keyboard (0.99.trunk.phablet2+15.04.20150209-0ubuntu1) vivid; urgency=medium

  [ Michael Sheldon ]
  * Add contractions as overrides (I've, you've, I'm, etc.). (LP:
    #1416020)
  * Recreate prediction databases with apostrophe support and remove
    Gutenberg license text from sample data. (LP: #1384800)
  * Always display the language button, even when only one language is
    enabled (so settings is easily accessible) and modify language
    button behaviour to toggle between most recently used layouts when
    tapped and to only show the menu when long pressed. (LP: #1417048,
    #1416037, #1416157)

  [ CI Train Bot ]
  * Resync trunk

  [ Michael Terry ]
  * Add em dash, en dash, and the bullet as extended hyphen options.
    (LP: #1414763)

  [ Ubuntu daily release ]
  * New rebuild forced

  [ Bill Filler ]
  * enable emoji by default (LP: #1416036)

  [ Penk Chen ]
  * Add mechanism for pure QML layouts to provide suggestion candidates.

  [ Mike Sheldon ]
  * Recreate prediction databases with apostrophe support and remove
    Gutenberg license text from sample data. (LP: #1384800)

ubuntu-keyboard (0.99.trunk.phablet2+15.04.20150202-0ubuntu1) vivid; urgency=medium

  [ Michael Sheldon ]
  * Enable auto-punctuation for Pinyin input. (LP: #1407588)
  * Speed up autopilot tests by only restarting maliit-server at the
    beginning and end of all tests
  * Restore a user's original input if they press backspace immediately
    after auto-completing a word. (LP: #1411645)
  * Allow Emoji keyboard to be shown for URL and email field types and
    switch out of Emoji layout when changing fields. (LP: #1415129,
    #1415127)
  * Adds a setting to allow the keyboard to remain hidden at all times
    (e.g. when a hardware keyboard is connected) (LP: #1415126)
  * Fix autocaps for clients that send multiline surrounding text (LP:
    #1415881)

  [ Ubuntu daily release ]
  * New rebuild forced

ubuntu-keyboard (0.99.trunk.phablet2+15.04.20150119-0ubuntu1) vivid; urgency=low

  [ Michael Sheldon ]
  * Deactivate autocaps if we receive find out we're in a field that
    doesn't support it after having already entered the field (LP:
    #1412460)
  * Add dependencies for any keyboard layouts used in autopilot tests
 -- CI Train Bot <email address hidden> Fri, 13 Feb 2015 20:33:15 +0000

Changed in ubuntu-keyboard (Ubuntu RTM):
status: New → Fix Released
Changed in canonical-devices-system-image:
status: In Progress → Fix Released
Changed in ubuntu-keyboard:
status: In Progress → Fix Released
Bill Filler (bfiller)
tags: added: ww09
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.