Comment #6 on issue 9830 by [email protected]: Backing store gets lost
http://code.google.com/p/chromium/issues/detail?id=9830
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=13725
------------------------------------------------------------------------
r13725 | [email protected] | 2009-04-14 17:52:44 -0700 (Tue, 14 Apr
2009) | 12 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/web_contents.cc?r1=13725&r2=13724
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/web_contents_view.h?r1=13725&r2=13724
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/web_contents_view_gtk.cc?r1=13725&r2=13724
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/web_contents_view_gtk.h?r1=13725&r2=13724
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/web_contents_view_mac.h?r1=13725&r2=13724
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/web_contents_view_mac.mm?r1=13725&r2=13724
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/web_contents_view_win.cc?r1=13725&r2=13724
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/web_contents_view_win.h?r1=13725&r2=13724
Fix missing backing store in Linux.
Issue: In the renderer host, the RenderWidgetHostView gets created before
the ViewMsg_New gets sent. In the Linux code, gtk packing of the
RenderWidgetHostView will cause a configure-event to happen, which leads to
ViewHost_Resize getting sent before the ViewMsg_New gets sent. This
eventually leads to the backing store never getting created. See the bug
report for details.
Fix:
Add a SetChildSize() method to WebContentsView.
Windows and Mac keep their current flow of just calling SetSize() on the
child.
Linux defers packing from the RenderWidgetHostView creation to the
SetChildSize() call.
BUG=9830
Review URL: http://codereview.chromium.org/67044
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
