diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-13 17:03:52 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-13 17:03:52 +0000 |
| commit | 04741728117ac420a3b4fd483a96a3a087e79b17 (patch) | |
| tree | e3265747f56d0a77c151bebae41f0d6ba8357aaa /android_webview/common/render_view_messages.h | |
| parent | 63e53c9725753134e65458fc8b61b313382dccb5 (diff) | |
| download | chromium_src-04741728117ac420a3b4fd483a96a3a087e79b17.zip chromium_src-04741728117ac420a3b4fd483a96a3a087e79b17.tar.gz chromium_src-04741728117ac420a3b4fd483a96a3a087e79b17.tar.bz2 | |
Fix race condition introduced in r242200 where AwContentsIoThreadClient is queried before a subframe's entry is added.
This happened because the entry was updated on the UI thread through WebContentsObserver::RenderFrameCreated. The race condition was that the IO thread could dispatch a resource request for that subframe before the map is updated.
The fix is to send a message from the renderer to the IO thread when a subframe is created so that the map is updated before any resouce requests arrive.
BUG=304341
R=mkosiba@chromium.org
Review URL: https://codereview.chromium.org/135443002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@244521 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'android_webview/common/render_view_messages.h')
| -rw-r--r-- | android_webview/common/render_view_messages.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/android_webview/common/render_view_messages.h b/android_webview/common/render_view_messages.h index 35266dd..4aefc35 100644 --- a/android_webview/common/render_view_messages.h +++ b/android_webview/common/render_view_messages.h @@ -110,3 +110,8 @@ IPC_SYNC_MESSAGE_CONTROL2_1(AwViewHostMsg_ShouldOverrideUrlLoading, int /* render_frame_id id */, base::string16 /* in - url */, bool /* out - result */) + +// Sent when a subframe is created. +IPC_MESSAGE_CONTROL2(AwViewHostMsg_SubFrameCreated, + int /* parent_render_frame_id */, + int /* child_render_frame_id */) |