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

Unified Diff: content/browser/renderer_host/render_widget_host_view_aura_unittest.cc

Issue 2773433003: Fix CompositorResizeLock to do something. (Closed)
Patch Set: resizelock: outlivescompositorandtimeoutbleeding Created 3 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/renderer_host/render_widget_host_view_aura_unittest.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc b/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
index 1a22959c5c6f239253b47855045529edf20cbc65..25c35819becc40de8c4509f4889b061594aad66b 100644
--- a/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
+++ b/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
@@ -47,7 +47,6 @@
#include "content/browser/renderer_host/render_widget_host_impl.h"
#include "content/browser/renderer_host/render_widget_host_view_event_handler.h"
#include "content/browser/renderer_host/render_widget_host_view_frame_subscriber.h"
-#include "content/browser/renderer_host/resize_lock.h"
#include "content/browser/renderer_host/text_input_manager.h"
#include "content/browser/web_contents/web_contents_view_aura.h"
#include "content/common/host_shared_bitmap_manager.h"
@@ -354,27 +353,12 @@ class FakeDelegatedFrameHostClientAura : public DelegatedFrameHostClientAura {
explicit FakeDelegatedFrameHostClientAura(
RenderWidgetHostViewAura* render_widget_host_view)
: DelegatedFrameHostClientAura(render_widget_host_view) {}
- ~FakeDelegatedFrameHostClientAura() override {}
+ ~FakeDelegatedFrameHostClientAura() override = default;
void DisableResizeLock() { can_create_resize_lock_ = false; }
private:
- // A lock that doesn't actually do anything to the compositor, and does not
- // time out.
- class FakeResizeLock : public ResizeLock {
- public:
- FakeResizeLock(const gfx::Size new_size, bool defer_compositor_lock)
- : ResizeLock(new_size, defer_compositor_lock) {}
- };
-
- // DelegatedFrameHostClientAura:
- std::unique_ptr<ResizeLock> DelegatedFrameHostCreateResizeLock(
- bool defer_compositor_lock) override {
- gfx::Size desired_size =
- render_widget_host_view()->GetNativeView()->bounds().size();
- return std::unique_ptr<ResizeLock>(
- new FakeResizeLock(desired_size, defer_compositor_lock));
- }
+ // DelegatedFrameHostClientAura implementation.
bool DelegatedFrameCanCreateResizeLock() const override {
return can_create_resize_lock_;
}
@@ -391,9 +375,8 @@ class FakeRenderWidgetHostViewAura : public RenderWidgetHostViewAura {
: RenderWidgetHostViewAura(widget, is_guest_view_hack),
delegated_frame_host_client_(
new FakeDelegatedFrameHostClientAura(this)) {
- std::unique_ptr<DelegatedFrameHostClient> client(
- delegated_frame_host_client_);
- InstallDelegatedFrameHostClient(this, std::move(client));
+ InstallDelegatedFrameHostClient(
+ this, base::WrapUnique(delegated_frame_host_client_));
}
~FakeRenderWidgetHostViewAura() override {}

Powered by Google App Engine
This is Rietveld 408576698