New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 686648 link

Starred by 1 user
Status: Verified
Owner:
█ msten...@opera.com
NOT IN USE
Closed: Jan 2017
Cc:
█ xlai@chromium.org
yigu@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Webkit source warnings. Unused variable pageLogicalHeight

Project Member Reported by llozano@chromium.org, Jan 30 2017 
we are seeing the following warning on the ChromeOS side.
Not sure why no one else has seen these. 

Please fix these ASAP.

../../third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp:1037:22: error  unused variable 'pageLogicalHeight' [-Werror,-Wunused-variable]
../../third_party/WebKit/Source/core/layout/LayoutBox.cpp:4750:18: error: unused variable 'pageLogicalHeight' [-Werror,-Wunused-variable]

the line number for the second one may have changed.

Not sure about which componet to assign to. cc'ing a few previous commiters to get ideas on who to assing.

Comment 1 by yigu@chromium.org, Jan 31 2017

Owner: msten...@opera.com
Status: Assigned (was: Untriaged)
It seems that both of them were added by mstensho@. PTAL.

Comment 2 by msten...@opera.com, Jan 31 2017

Status: Fixed (was: Assigned)

Comment 3 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 4 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 5 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 6 by abod...@chromium.org, Aug 3 2017

Status: Verified (was: Fixed)
Closing. Please reopen it if its not fixed. Thanks!
►

Sign in to add a comment