New issue
Advanced search Search tips

Issue 816807 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : Text box appears chopped from bottom after clicking on File I/O option in Pnacl.

Reported by rp...@etouch.net, Feb 27 2018

Issue description

Version: 66.0.3355.0 (Official Build) Revision31d0bb0369f909177f2ce90b439f296bcf9c90cb-refs/heads/master@{#539001}(32/64-bit)
OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.12.6,10.13.1,10.13.4)
URL : https://chrome.google.com/webstore/detail/pnacl-examples/mblemkccghnfkjignlmgngmopopifacf?utm_source=chrome-ntp-icon

What steps will reproduce the problem?
1. Launch chrome,navigate to above url and click on 'Add to chrome' to install and launch app.
2. Now click on File I/O option and observe bottom area of large text box on RHS 
 
Actual: Text box appears chopped from bottom after clicking on File I/O option 
Expected: Text box shouldn't appear chopped from bottom after clicking on File I/O option

This is regression issue, broken in ‘M 61’ and below is the bisect info :
Good build: 61.0.3137.0  (Revision: 481057).
Bad build: 61.0.3138.0 (Revision: 481386).

You are probably looking for a change made after 481207 (known good), but no later than 481208 (first known bad).

CHANGELOG URL:

The script might not always return single CL as suspect as some perf builds might get missing due to failure.

https://chromium.googlesource.com/chromium/src/+log/148c2697ed43abbc96e517b0c65517ebcef171b7..4cef659a8009c5cb4f5708336ae37654d23653b3

Suspect : https://chromium.googlesource.com/chromium/src/+/4cef659a8009c5cb4f5708336ae37654d23653b3

From the CL above, assigning the issue to the concern owner 

@dtapuska- Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Note : The above issue is also seen on Stable build #64.0.3282.186,Beta build #65.0.3325.88 and Dev build #66.0.3350.0


Thanks!
 
Actual_video.mov
1.8 MB View Download
Expected_video.mov
1.7 MB View Download

Comment 1 by rp...@etouch.net, Feb 27 2018

Labels: RegressedIn-61 Target-65 FoundIn-66 Target-66 FoundIn-64 FoundIn-65 Target-64
Status: WontFix (was: Assigned)
The extension needs to be updated it is assuming the scrolling element is the body which isn't true anymore.

Specifically this code:
    var iframeDocument = this.contentWindow.document;
    var iframeBodyEl = iframeDocument.body;
    iframeEl.style.height = iframeBodyEl.scrollHeight + 'px';

is consuming the scrollHeight from the body element and not the scrollingElement.


Sign in to add a comment