Commit 2eb574ce authored by Dongheun Kang's avatar Dongheun Kang Committed by Commit Bot

Remove unused variables in local_frame_view.cc

Bug: n/a

Change-Id: Iffd7f0bece86b9438b285da3a53ceea00104b77d
Reviewed-on: https://chromium-review.googlesource.com/1195383
Commit-Queue: Stefan Zager <szager@chromium.org>
Reviewed-by: default avatarStefan Zager <szager@chromium.org>
Reviewed-by: default avatarDavid Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#587936}
parent 61e0b329
...@@ -3318,7 +3318,7 @@ LayoutPoint LocalFrameView::ConvertToContainingEmbeddedContentView( ...@@ -3318,7 +3318,7 @@ LayoutPoint LocalFrameView::ConvertToContainingEmbeddedContentView(
FloatPoint LocalFrameView::ConvertToContainingEmbeddedContentView( FloatPoint LocalFrameView::ConvertToContainingEmbeddedContentView(
const FloatPoint& local_point) const { const FloatPoint& local_point) const {
if (LocalFrameView* parent = ParentFrameView()) { if (ParentFrameView()) {
auto* layout_object = frame_->OwnerLayoutObject(); auto* layout_object = frame_->OwnerLayoutObject();
if (!layout_object) if (!layout_object)
return local_point; return local_point;
...@@ -3476,10 +3476,8 @@ void LocalFrameView::AddScrollableArea( ...@@ -3476,10 +3476,8 @@ void LocalFrameView::AddScrollableArea(
scrollable_areas_ = new ScrollableAreaSet; scrollable_areas_ = new ScrollableAreaSet;
scrollable_areas_->insert(scrollable_area); scrollable_areas_->insert(scrollable_area);
if (ScrollingCoordinator* scrolling_coordinator = if (GetScrollingCoordinator())
this->GetScrollingCoordinator()) {
ScrollableAreasDidChange(); ScrollableAreasDidChange();
}
} }
void LocalFrameView::RemoveScrollableArea( void LocalFrameView::RemoveScrollableArea(
...@@ -3488,10 +3486,8 @@ void LocalFrameView::RemoveScrollableArea( ...@@ -3488,10 +3486,8 @@ void LocalFrameView::RemoveScrollableArea(
return; return;
scrollable_areas_->erase(scrollable_area); scrollable_areas_->erase(scrollable_area);
if (ScrollingCoordinator* scrolling_coordinator = if (GetScrollingCoordinator())
this->GetScrollingCoordinator()) {
ScrollableAreasDidChange(); ScrollableAreasDidChange();
}
} }
void LocalFrameView::AddAnimatingScrollableArea( void LocalFrameView::AddAnimatingScrollableArea(
...@@ -3889,10 +3885,8 @@ void LocalFrameView::SetSelfVisible(bool visible) { ...@@ -3889,10 +3885,8 @@ void LocalFrameView::SetSelfVisible(bool visible) {
void LocalFrameView::Show() { void LocalFrameView::Show() {
if (!IsSelfVisible()) { if (!IsSelfVisible()) {
SetSelfVisible(true); SetSelfVisible(true);
if (ScrollingCoordinator* scrolling_coordinator = if (GetScrollingCoordinator())
this->GetScrollingCoordinator()) {
GetScrollingContext()->SetScrollGestureRegionIsDirty(true); GetScrollingContext()->SetScrollGestureRegionIsDirty(true);
}
SetNeedsCompositingUpdate(kCompositingUpdateRebuildTree); SetNeedsCompositingUpdate(kCompositingUpdateRebuildTree);
if (IsParentVisible()) { if (IsParentVisible()) {
ForAllChildViewsAndPlugins( ForAllChildViewsAndPlugins(
...@@ -3912,10 +3906,8 @@ void LocalFrameView::Hide() { ...@@ -3912,10 +3906,8 @@ void LocalFrameView::Hide() {
}); });
} }
SetSelfVisible(false); SetSelfVisible(false);
if (ScrollingCoordinator* scrolling_coordinator = if (GetScrollingCoordinator())
this->GetScrollingCoordinator()) {
GetScrollingContext()->SetScrollGestureRegionIsDirty(true); GetScrollingContext()->SetScrollGestureRegionIsDirty(true);
}
SetNeedsCompositingUpdate(kCompositingUpdateRebuildTree); SetNeedsCompositingUpdate(kCompositingUpdateRebuildTree);
} }
} }
......
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