Commit d6ec8154 authored by Christopher Cameron's avatar Christopher Cameron Committed by Commit Bot

Enable FlingSchedulerMac

Enable tests that were disabled because of this functionality missing.

Bug: 842325
Change-Id: I6f540460f4e5f81d7ff8dd36e4abf369f0bd4785
Reviewed-on: https://chromium-review.googlesource.com/1149612Reviewed-by: default avatarJonathan Ross <jonross@chromium.org>
Commit-Queue: ccameron <ccameron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577957}
parent da08e180
...@@ -148,7 +148,7 @@ class CONTENT_EXPORT BrowserCompositorMac : public DelegatedFrameHostClient, ...@@ -148,7 +148,7 @@ class CONTENT_EXPORT BrowserCompositorMac : public DelegatedFrameHostClient,
bool ForceNewSurfaceForTesting(); bool ForceNewSurfaceForTesting();
ui::Compositor* GetCompositorForTesting() const; ui::Compositor* GetCompositor() const;
private: private:
// ui::LayerObserver implementation: // ui::LayerObserver implementation:
......
...@@ -487,7 +487,9 @@ void BrowserCompositorMac::LayerDestroyed(ui::Layer* layer) { ...@@ -487,7 +487,9 @@ void BrowserCompositorMac::LayerDestroyed(ui::Layer* layer) {
SetParentUiLayer(nullptr); SetParentUiLayer(nullptr);
} }
ui::Compositor* BrowserCompositorMac::GetCompositorForTesting() const { ui::Compositor* BrowserCompositorMac::GetCompositor() const {
if (parent_ui_layer_)
return parent_ui_layer_->GetCompositor();
if (recyclable_compositor_) if (recyclable_compositor_)
return recyclable_compositor_->compositor(); return recyclable_compositor_->compositor();
return nullptr; return nullptr;
......
...@@ -21,14 +21,10 @@ ui::Compositor* FlingSchedulerMac::GetCompositor() { ...@@ -21,14 +21,10 @@ ui::Compositor* FlingSchedulerMac::GetCompositor() {
// RWHV_child_frame doesn't have DelegatedFrameHost with ui::Compositor. // RWHV_child_frame doesn't have DelegatedFrameHost with ui::Compositor.
if (host_->GetView()->IsRenderWidgetHostViewChildFrame()) if (host_->GetView()->IsRenderWidgetHostViewChildFrame())
return nullptr; return nullptr;
RenderWidgetHostViewMac* view =
// TODO(sahel): Uncomment this once Viz is ready on Mac.
// https://crbug.com/833985
/* RenderWidgetHostViewMac* view =
static_cast<RenderWidgetHostViewMac*>(host_->GetView()); static_cast<RenderWidgetHostViewMac*>(host_->GetView());
if (view->BrowserCompositor()) if (view->BrowserCompositor())
return view->BrowserCompositor()->Compositor(); return view->BrowserCompositor()->GetCompositor();
} */
return nullptr; return nullptr;
} }
......
...@@ -1837,11 +1837,11 @@ TEST_F(InputMethodMacTest, MonitorCompositionRangeForActiveWidget) { ...@@ -1837,11 +1837,11 @@ TEST_F(InputMethodMacTest, MonitorCompositionRangeForActiveWidget) {
TEST_F(RenderWidgetHostViewMacTest, ClearCompositorFrame) { TEST_F(RenderWidgetHostViewMacTest, ClearCompositorFrame) {
BrowserCompositorMac* browser_compositor = rwhv_mac_->BrowserCompositor(); BrowserCompositorMac* browser_compositor = rwhv_mac_->BrowserCompositor();
ui::Compositor* ui_compositor = browser_compositor->GetCompositorForTesting(); ui::Compositor* ui_compositor = browser_compositor->GetCompositor();
EXPECT_NE(ui_compositor, nullptr); EXPECT_NE(ui_compositor, nullptr);
EXPECT_TRUE(ui_compositor->IsLocked()); EXPECT_TRUE(ui_compositor->IsLocked());
rwhv_mac_->ClearCompositorFrame(); rwhv_mac_->ClearCompositorFrame();
EXPECT_EQ(browser_compositor->GetCompositorForTesting(), ui_compositor); EXPECT_EQ(browser_compositor->GetCompositor(), ui_compositor);
EXPECT_FALSE(ui_compositor->IsLocked()); EXPECT_FALSE(ui_compositor->IsLocked());
} }
......
### Mac ### Mac
# BrowserSideFlingBrowserTest timing out https://crbug.com/842325
-BrowserSideFlingBrowserTest.AutoscrollFling
-BrowserSideFlingBrowserTest.TouchpadFling
-BrowserSideFlingBrowserTest.TouchscreenFling
-SitePerProcessBrowserTest.TouchpadGestureFlingStart
-SitePerProcessBrowserTest.TouchscreenGestureFlingStart
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment