• mukai@chromium.org's avatar
    Add EventFilter for LauncherTooltip (2nd try). · 12c05943
    mukai@chromium.org authored
    Now it monitors mouse/touch/gesture events and hides the tooltip if necessary.
    Previous CL (crrev.com/148079) was reverted since it causes use-after-free error.
    It happens when the 'LauncherTooltipBubble' widget exists but LauncherTooltipManager
    is already released.  In that case, the widget's release will cause WindowClosing()
    which calls host_->OnBubbleClosed(), but host_ is already released.
    Thus I'd make sure to introduce a method to set host_ to NULL in case that
    LauncherTooltipManager explicitly clean-up the Bubble.
    
    R=derat@chromium.org
    BUG=137678
    TEST=manually checked on linux, aura_shell_unittests passed with asan
    
    
    Review URL: https://chromiumcodereview.appspot.com/10808102
    
    git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148308 0039d316-1c4b-4281-b951-d872f2087c98
    12c05943
launcher_view.cc 35 KB