diff options
author | Zeno Albisser <zeno.albisser@digia.com> | 2013-11-21 14:09:57 +0100 |
---|---|---|
committer | Andras Becsi <andras.becsi@digia.com> | 2013-11-29 15:14:36 +0100 |
commit | eb32ba6f51d0c21d58cd7d89785285ff8fa64624 (patch) | |
tree | 2c7c940e1dbee81b89d935626110816b494aa32c /chromium/content/browser/browser_plugin/browser_plugin_guest.cc | |
parent | 9427c1a0222ebd67efef1a2c7990a0fa5c9aac84 (diff) | |
download | qtwebengine-chromium-eb32ba6f51d0c21d58cd7d89785285ff8fa64624.tar.gz |
Update chromium to branch 1599.
Change-Id: I04e775a946a208bb4500d3b722bcb05c82b9d7cb
Reviewed-by: Andras Becsi <andras.becsi@digia.com>
Diffstat (limited to 'chromium/content/browser/browser_plugin/browser_plugin_guest.cc')
-rw-r--r-- | chromium/content/browser/browser_plugin/browser_plugin_guest.cc | 28 |
1 files changed, 4 insertions, 24 deletions
diff --git a/chromium/content/browser/browser_plugin/browser_plugin_guest.cc b/chromium/content/browser/browser_plugin/browser_plugin_guest.cc index 31250b7d626..5f1ec69e8af 100644 --- a/chromium/content/browser/browser_plugin/browser_plugin_guest.cc +++ b/chromium/content/browser/browser_plugin/browser_plugin_guest.cc @@ -127,7 +127,7 @@ class BrowserPluginGuest::GeolocationRequest : public PermissionRequest { // in the fact whether the embedder/app has geolocation // permission. Therefore we use an invalid |bridge_id|. -1 /* bridge_id */, - web_contents->GetLastCommittedURL(), + web_contents->GetURL(), geolocation_callback); return; } @@ -348,7 +348,7 @@ BrowserPluginGuest::BrowserPluginGuest( embedder_visible_(true), next_permission_request_id_(browser_plugin::kInvalidPermissionRequestID), has_render_view_(has_render_view), - last_seen_auto_size_enabled_(false) { + is_in_destruction_(false) { DCHECK(web_contents); web_contents->SetDelegate(this); if (opener) @@ -422,6 +422,7 @@ int BrowserPluginGuest::RequestPermission( } void BrowserPluginGuest::Destroy() { + is_in_destruction_ = true; if (!attached() && opener()) opener()->pending_new_windows_.erase(this); DestroyUnattachedWindows(); @@ -765,10 +766,6 @@ WebContentsImpl* BrowserPluginGuest::GetWebContents() { base::SharedMemory* BrowserPluginGuest::GetDamageBufferFromEmbedder( const BrowserPluginHostMsg_ResizeGuest_Params& params) { - if (!attached()) { - LOG(WARNING) << "Attempting to map a damage buffer prior to attachment."; - return NULL; - } #if defined(OS_WIN) base::ProcessHandle handle = embedder_web_contents_->GetRenderProcessHost()->GetHandle(); @@ -1264,7 +1261,7 @@ void BrowserPluginGuest::OnLockMouse(bool user_gesture, base::Value::CreateBooleanValue(last_unlocked_by_target)); request_info.Set(browser_plugin::kURL, base::Value::CreateStringValue( - web_contents()->GetLastCommittedURL().spec())); + web_contents()->GetURL().spec())); RequestPermission(BROWSER_PLUGIN_PERMISSION_TYPE_POINTER_LOCK, new PointerLockRequest(this), @@ -1321,13 +1318,6 @@ void BrowserPluginGuest::OnResizeGuest( render_widget_host->NotifyScreenInfoChanged(); } } - // When autosize is turned off and as a result there is a layout change, we - // send a sizechanged event. - if (!auto_size_enabled_ && last_seen_auto_size_enabled_ && - !params.view_rect.size().IsEmpty() && delegate_) { - delegate_->SizeChanged(last_seen_view_size_, params.view_rect.size()); - last_seen_auto_size_enabled_ = false; - } // Invalid damage buffer means we are in HW compositing mode, // so just resize the WebContents and repaint if needed. if (!base::SharedMemory::IsHandleValid(params.damage_buffer_handle)) { @@ -1590,16 +1580,6 @@ void BrowserPluginGuest::OnUpdateRect( params.flags); relay_params.needs_ack = params.needs_ack; - bool size_changed = last_seen_view_size_ != params.view_size; - gfx::Size old_size = last_seen_view_size_; - last_seen_view_size_ = params.view_size; - - if ((auto_size_enabled_ || last_seen_auto_size_enabled_) && - size_changed && delegate_) { - delegate_->SizeChanged(old_size, last_seen_view_size_); - } - last_seen_auto_size_enabled_ = auto_size_enabled_; - // HW accelerated case, acknowledge resize only if (!params.needs_ack || !damage_buffer_) { relay_params.damage_buffer_sequence_id = 0; |