diff options
| author | willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-08 23:14:39 +0000 |
|---|---|---|
| committer | willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-08 23:14:39 +0000 |
| commit | 18beec52826496c4098e243afd64cba7d1fb0959 (patch) | |
| tree | 67a08bcbe5e1a42c7277d6b066dfab641db040ac /tools | |
| parent | 4a0765a6e06b2e188c68999a52b3fea976e72c40 (diff) | |
| download | chromium_src-18beec52826496c4098e243afd64cba7d1fb0959.zip chromium_src-18beec52826496c4098e243afd64cba7d1fb0959.tar.gz chromium_src-18beec52826496c4098e243afd64cba7d1fb0959.tar.bz2 | |
Stop leaking RenderWidgetHostViewGtk.
Don't bother calling gfx::RemoveAllChildren() in TabContentsViewGtk when creating a new RWHV, since any previously existing RWHV widget should already have been deleted.
This also fixes interstitial pages crashing. Although instead we have two RWHV widgets in the same TabContentsView vbox :) I'll fix that in a later changelist.
Also update the valgrind suppressions file.
BUG=http://www.crbug.com/10747
BUG=http://www.crbug.com/8486
Review URL: http://codereview.chromium.org/115145
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15695 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/valgrind/suppressions.txt | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/tools/valgrind/suppressions.txt b/tools/valgrind/suppressions.txt index 6df51bf..4df5902 100644 --- a/tools/valgrind/suppressions.txt +++ b/tools/valgrind/suppressions.txt @@ -439,21 +439,6 @@ fun:_ZN22RendererSecurityPolicy15GrantRequestURLEiRK4GURL } { - # see http://crbug.com/10747 - bug_10747 - Memcheck:Leak - fun:_Znwj - fun:_ZN18WebContentsViewGtk19CreateViewForWidgetEP16RenderWidgetHost - fun:_ZN11WebContents32CreateRenderViewForRenderManagerEP14RenderViewHost -} -{ - # one more from http://crbug.com/10747 - bug_10747b - Memcheck:Leak - fun:_Znwj - fun:_ZN18TabContentsViewGtk19CreateViewForWidgetEP16RenderWidgetHost -} -{ # see http://crbug.com/10748 bug_10748 Memcheck:Leak |