Bug 43648 - [Chromium] Notify the WebFrameClient when the frame is detached from the view.
Summary: [Chromium] Notify the WebFrameClient when the frame is detached from the view.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 43775 43909
Blocks:
  Show dependency tree / graph
 
Reported: 2010-08-06 15:33 PDT by James Hawkins
Modified: 2010-08-12 05:26 PDT (History)
5 users (show)

See Also:

Attachments
Patch (2.14 KB, patch)
2010-08-06 15:33 PDT, James Hawkins 		no flags Details | Formatted Diff | Diff
Patch (2.33 KB, patch)
2010-08-06 15:38 PDT, James Hawkins 		no flags Details | Formatted Diff | Diff
Patch (2.40 KB, patch)
2010-08-11 11:50 PDT, James Hawkins 		dimich: review+
Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description James Hawkins 2010-08-06 15:33:19 PDT 
[Chromium] Notify the WebFrameClient when the frame is detached from the view.
Comment 1 James Hawkins 2010-08-06 15:33:57 PDT 
Created attachment 63772 [details]
Patch
Comment 2 James Hawkins 2010-08-06 15:38:28 PDT 
Created attachment 63773 [details]
Patch
Comment 3 Dimitri Glazkov (Google) 2010-08-06 15:42:13 PDT 
Comment on attachment 63773 [details]
Patch

ok.
Comment 4 James Hawkins 2010-08-09 10:06:09 PDT 
Committed r64988: <http://trac.webkit.org/changeset/64988>
Comment 5 James Hawkins 2010-08-11 11:48:29 PDT 
This commit was rolled out due to crashes in the Chromium Worker UI tests.
Comment 6 James Hawkins 2010-08-11 11:50:41 PDT 
Created attachment 64141 [details]
Patch
Comment 7 Dimitri Glazkov (Google) 2010-08-11 11:57:31 PDT 
Dimich, David, can you take a look at this patch? I am not sure what we're fixing, so I would seek your opinion. Apparently, the crashes came from the worker tests.
Comment 8 Dmitry Titov 2010-08-11 12:42:35 PDT 
Comment on attachment 64141 [details]
Patch

r=me
Comment 9 James Hawkins 2010-08-11 13:11:06 PDT 
Committed r65181: <http://trac.webkit.org/changeset/65181>