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

Issue 691142 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

the ɴᴀᴄʟ devenv does no longer on newer google chrome versions for ᴀᴍᴅ⁶⁴

Reported by lael.cel...@gmail.com, Feb 10 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.37 Safari/537.36

Steps to reproduce the problem:
Launch the app with native client enable chrome

What went wrong?
the app’s window disappear before the command prompt is available

WebStore page: https://chrome.google.com/webstore/detail/nacl-development-environm/aljpgkjeipgnmdpikaajmnepbcfkglfa

Did this work before? Yes 54.0.2840.71

Chrome version: 57.0.2987.37  Channel: beta

I tried it on Linux Windows® and ᴏꜱX.
The exact same problem affect https://chrome.google.com/webstore/detail/python/nodpmmidbgeganfponihbgmfcoiibffi

while https://chrome.google.com/webstore/detail/ruby-irb/mdoifndjjoblccpefkbdicgahkbbcfdk works.
 
Cc: ranjitkan@chromium.org
Labels: hasbisect-per-revision M-58 OS-Linux OS-Mac
Owner: jochen@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue and is broken in M55. Below is the bisect details obtained using per revision bisect:

Bisect Info:
============
55.0.2843.0 - Good build
55.0.2844.0 - Bad build

Bisect URL: https://chromium.googlesource.com/chromium/src/+log/83c73f2c78bbdec7b5e335b04c8f1aba2a0a07e5..7f8af00c5e56381925e4ca11fab8a2965151eb1c
Change URL: https://chromium.googlesource.com/chromium/src/+/7f8af00c5e56381925e4ca11fab8a2965151eb1c

@ jochen: Assigning to you, request you to please take a look into it.

Issue is observed on Windows 10, MAC 10.12.2 and Ubuntu 14.04 OS

Thanks.!

Comment 2 by jochen@chromium.org, Feb 15 2017

Cc: -ranjitkan@chromium.org
Owner: ranjitkan@chromium.org
The regression introduced by this CL is fixed by 8a4fe2fc28bb45f58445d5802057efdafbe7719b which was merged to 56 and 57
Cc: ranjitkan@chromium.org kkaluri@chromium.org
Labels: Needs-Feedback
Owner: jochen@chromium.org
Able to reproduce this issue on Mac 10.12.3 with chrome canary #58.0.3019.0

attaching the screen-cast for reference
jochen@ could you please look into it and let us know your observations.

Thank You...
Issue 691142.mp4
864 KB View Download
Able to reproduce this issue on Mac 10.12.3 with chrome canary #58.0.3026.0

jochen@ could you please provide an latest update on this issue.

Thank You...
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7acd424e3b6884424a4ab9df5ca9c8ab0f5a074d

commit 7acd424e3b6884424a4ab9df5ca9c8ab0f5a074d
Author: jochen <jochen@chromium.org>
Date: Tue Mar 07 17:18:27 2017

Ensure that we can repeatedly set an expando on a plugin object

Just checking whether the holder has the property in the setter
interceptor doesn't work, as this will invoke the get interceptor.

BUG= 691142 
R=haraken@chromium.org

Review-Url: https://codereview.chromium.org/2736783002
Cr-Commit-Position: refs/heads/master@{#455109}

[modify] https://crrev.com/7acd424e3b6884424a4ab9df5ca9c8ab0f5a074d/third_party/WebKit/LayoutTests/plugins/simple-expando.html
[modify] https://crrev.com/7acd424e3b6884424a4ab9df5ca9c8ab0f5a074d/third_party/WebKit/Source/bindings/core/v8/custom/V8HTMLPlugInElementCustom.cpp

Status: Fixed (was: Assigned)
Labels: Merge-Request-58
Labels: -Needs-Feedback ReleaseBlock-Stable
Project Member

Comment 9 by sheriffbot@chromium.org, Mar 8 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), bhthompson@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
We are planning to cut Dev RC today (03/08) at 5.00 PM PST. Please merge the CL to M58 branch (3029) ASAP.
Project Member

Comment 11 by bugdroid1@chromium.org, Mar 9 2017

Labels: -merge-approved-58 merge-merged-3029
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bd3dc76cf5f0040beefa918e56277e2867294c4c

commit bd3dc76cf5f0040beefa918e56277e2867294c4c
Author: Jochen Eisinger <jochen@chromium.org>
Date: Thu Mar 09 08:02:58 2017

Ensure that we can repeatedly set an expando on a plugin object

Just checking whether the holder has the property in the setter
interceptor doesn't work, as this will invoke the get interceptor.

BUG= 691142 
TBR=haraken@chromium.org

Review-Url: https://codereview.chromium.org/2736783002
Cr-Commit-Position: refs/heads/master@{#455109}
(cherry picked from commit 7acd424e3b6884424a4ab9df5ca9c8ab0f5a074d)

Review-Url: https://codereview.chromium.org/2745483002 .
Cr-Commit-Position: refs/branch-heads/3029@{#81}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/bd3dc76cf5f0040beefa918e56277e2867294c4c/third_party/WebKit/LayoutTests/plugins/simple-expando.html
[modify] https://crrev.com/bd3dc76cf5f0040beefa918e56277e2867294c4c/third_party/WebKit/Source/bindings/core/v8/custom/V8HTMLPlugInElementCustom.cpp

Sign in to add a comment