Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(260)

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 12881005: Allow CopyFromBackingStore to fallback to copying from the renderer side if the accelerated surface… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: unused include Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 747dcf04be9c75b8a3735805bbe926432c356b10..94225f50a9428b64fc81ac3dc87baa8da580e71f 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -636,15 +636,6 @@ WebPreferences WebContentsImpl::GetWebkitPrefs(RenderViewHost* rvh,
prefs.accelerated_compositing_enabled = false;
}
-#if defined(OS_LINUX) && !defined(USE_AURA)
- // Temporary fix for Linux non-Aura capturing. http://crbug.com/174957
- WebContentsImpl* web_contents =
- static_cast<WebContentsImpl*>(WebContents::FromRenderViewHost(rvh));
- if (web_contents && web_contents->capturer_count_ > 0) {
- prefs.accelerated_compositing_enabled = false;
- }
-#endif
-
return prefs;
}
@@ -991,16 +982,6 @@ void WebContentsImpl::IncrementCapturerCount() {
++capturer_count_;
DVLOG(1) << "There are now " << capturer_count_
<< " capturing(s) of WebContentsImpl@" << this;
-
-#if defined(OS_LINUX) && !defined(USE_AURA)
- // Temporary fix for Linux non-Aura capturing. http://crbug.com/174957
- if (capturer_count_ == 1) {
- // Force a WebkitPreferences reload to disable compositing for snapshots.
- RenderViewHost* rvh = GetRenderViewHost();
- if (rvh)
- rvh->UpdateWebkitPreferences(rvh->GetWebkitPreferences());
- }
-#endif
}
void WebContentsImpl::DecrementCapturerCount() {
@@ -1012,16 +993,6 @@ void WebContentsImpl::DecrementCapturerCount() {
if (is_being_destroyed_)
return;
-#if defined(OS_LINUX) && !defined(USE_AURA)
- // Temporary fix for Linux non-Aura capturing. http://crbug.com/174957
- if (capturer_count_ == 0) {
- // Force a WebkitPreferences reload to re-enable compositing.
- RenderViewHost* rvh = GetRenderViewHost();
- if (rvh)
- rvh->UpdateWebkitPreferences(rvh->GetWebkitPreferences());
- }
-#endif
-
// While capturer_count_ was greater than zero, the WasHidden() calls to RWHV
// were being prevented. If there are no more capturers, make the call now.
if (capturer_count_ == 0 && !should_normally_be_visible_) {

Powered by Google App Engine
This is Rietveld 408576698