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

Issue 724920 link

Starred by 6 users
Status: Verified
Owner:
wangxianzhu@chromium.org
Closed: May 2017
Cc:
ajha@chromium.org
█ manoranj...@chromium.org
█ kavvaru@chromium.org
wangxianzhu@chromium.org
nyerramilli@chromium.org
█ bustamante@chromium.org
█ n...@chromium.org
rbasuvula@chromium.org
█ brajkumar@chromium.org
dbeam@chromium.org
█ msrchandra@chromium.org
valdrin@google.com
█ ranjitkan@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 1
Type: Bug-Regression

Blocking:
issue 724263



Sign in to add a comment

Regression:Delay is seen while opening the sub page in language section of Md-settings.

Project Member Reported by rkalavakuntla@chromium.org, May 22 2017 
Chrome Version:60.0.3105.0/9574.0.0 dev-channel Candy,Peppy,Minnie
OS:Chrome

What steps will reproduce the problem?
(1)Sign in to user->Open chrome://md-settings/ page and go near Languages section
(2)Click on arrow icon and Observe

Actual:Delay is seen while opening the sub page
Expected:Delay shouldn't be seen while opening the sub page.

This is a Regression issue broken in M-60

Manual Bisect Info:
--------------------
Good build:60.0.3104.0
Bad build :60.0.3105.0

Other info should be updated soon

Comment 1 by rkalavakuntla@chromium.org, May 22 2017 

Issue is also seen in Windows 7 and Ubuntu 14.04,checked with version #60.0.3105.0 dev
Note:Issue is also seen in other places like Settings menu..
Actualdelay.webm
2.3 MB View Download
Expected.ogv
2.8 MB View Download

Comment 2 by rkalavakuntla@chromium.org, May 22 2017

Labels: hasbisect-per-revision ReleaseBlock-Stable OS-Mac
Owner: wangxianzhu@chromium.org
Status: Assigned (was: Untriaged)
Able to reproduce the issue on Windows 7,Ubuntu 14.04 & Mac 10.12.4 using latest Canary #60.0.3105.0 dev

Manual Bisect Info:
-------------------
Good Build:60.0.3104.0-Revision-473014
Bad Build :60.0.3105.0-Revision-473393

Using per-revision bisect providing the bisect results:

You are probably looking for a change made after 473370 (known good), but no later than 473371 (first known bad).
CHANGELOG URL:
 https://chromium.googlesource.com/chromium/src/+log/d00385ad4d2d12bf0aede04cbe08ae4122148ed7..3c7f1c4ecd1421d074daa5a103ea3f895c1c8724

Suspecting: https://chromium.googlesource.com/chromium/src/+/3c7f1c4ecd1421d074daa5a103ea3f895c1c8724
Review-Url: https://codereview.chromium.org/2894093002

wangxianzhu@:Could you please take a look into this issue and reassign if this issue is not related to your change.

Thanks..!!

Comment 3 by ajha@chromium.org, May 22 2017

Cc: msrchandra@chromium.org wangxianzhu@chromium.org ranjitkan@chromium.org rbasuvula@chromium.org nyerramilli@chromium.org
 Issue 724945  has been merged into this issue.

Comment 4 by michae...@chromium.org, May 22 2017

Cc: dbeam@chromium.org
This is a dom-repeat, not an iron-list, and it's not created on-demand when opening the iron-collapse. So no rendering delay is expected here from an HTML/JS perspective.

#1: What do you mean by "Issue is also seen in other places like Settings menu.."? Is this referring to the nav menu on the left-hand side of the Settings page?

Comment 5 by michae...@chromium.org, May 22 2017

Cc: n...@chromium.org valdrin@google.com
+iron-collapse folks from Polymer for FYI

Comment 6 by wangxianzhu@chromium.org, May 22 2017 

 Issue 724936  has been merged into this issue.

Comment 7 by wangxianzhu@chromium.org, May 22 2017

Cc: manoranj...@chromium.org bustamante@chromium.org
 Issue 725041  has been merged into this issue.

Comment 8 by michae...@chromium.org, May 22 2017 

#1: Never mind, we have enough other bug reports of this. Thanks!
Project Member

Comment 9 by bugdroid1@chromium.org, May 22 2017 

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

commit 86c54d1018d43154254a9b479f43621bd8f6cad6
Author: wangxianzhu <wangxianzhu@chromium.org>
Date: Mon May 22 18:49:59 2017

Revert of Don't access DisplayItemClient::VisualRect() for cached display items. (patchset #3 id:40001 of https://codereview.chromium.org/2894093002/ )

Reason for revert:
BUG= 724920 ,724263, 725041 , 724936 

Original issue's description:
> Don't access DisplayItemClient::VisualRect() for cached display items.
>
> - Access DisplayItemClient::VisualRect() when a new DisplayItem is
>   created, and save it in the new DisplayItem.
> - When a cached display item is copied, don't access DisplayItemClient::
>   VisualRect() but fetch from the cached DisplayItem.
> - Remove DisplayItemList::visual_rects_.
> - Save original visual rects (LayoutRect) and apply GraphicsLayer offset
>   with EnclosingIntRect() when the display items are added into
>   WebDisplayItemList.
>
> Please see the bug for the reason.
>
> BUG=724263
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
>
> Review-Url: https://codereview.chromium.org/2894093002
> Cr-Commit-Position: refs/heads/master@{#473371}
> Committed: https://chromium.googlesource.com/chromium/src/+/3c7f1c4ecd1421d074daa5a103ea3f895c1c8724

TBR=chrishtr@chromium.org,wkorman@chromium.org,pdr@chromium.org,michaelpg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=724263

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

[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/core/frame/FrameView.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/core/paint/PaintControllerPaintTest.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/BUILD.gn
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/ContentLayerDelegate.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/ClipDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/ClipDisplayItem.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/ClipPathDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/ClipPathDisplayItem.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/CompositingDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/CompositingDisplayItem.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/DisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/DisplayItem.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/DisplayItemClient.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/DisplayItemList.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/DisplayItemList.h
[add] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/DisplayItemListTest.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/DrawingDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/DrawingDisplayItem.h
[delete] https://crrev.com/cabeb0d9ab1b39fc25a1647d38afade5bea2e70e/third_party/WebKit/Source/platform/graphics/paint/DrawingDisplayItemTest.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/FilterDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/FilterDisplayItem.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/FloatClipDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/FloatClipDisplayItem.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/ForeignLayerDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/ForeignLayerDisplayItem.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/PaintArtifact.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/PaintArtifact.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/PaintChunkerTest.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/PaintController.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/PaintController.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/ScrollDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/ScrollDisplayItem.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/Transform3DDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/Transform3DDisplayItem.h
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/TransformDisplayItem.cpp
[modify] https://crrev.com/86c54d1018d43154254a9b479f43621bd8f6cad6/third_party/WebKit/Source/platform/graphics/paint/TransformDisplayItem.h
Project Member

Comment 10 by bugdroid1@chromium.org, May 23 2017

Labels: merge-merged-3107
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/238659de163326395f9628bfc9cac04b55678f88

commit 238659de163326395f9628bfc9cac04b55678f88
Author: wangxianzhu <wangxianzhu@chromium.org>
Date: Tue May 23 00:15:31 2017

Revert of Don't access DisplayItemClient::VisualRect() for cached display items. (patchset #3 id:40001 of https://codereview.chromium.org/2894093002/ )

Reason for revert:
BUG= 724920 ,724263, 725041 , 724936 

Original issue's description:
> Don't access DisplayItemClient::VisualRect() for cached display items.
>
> - Access DisplayItemClient::VisualRect() when a new DisplayItem is
>   created, and save it in the new DisplayItem.
> - When a cached display item is copied, don't access DisplayItemClient::
>   VisualRect() but fetch from the cached DisplayItem.
> - Remove DisplayItemList::visual_rects_.
> - Save original visual rects (LayoutRect) and apply GraphicsLayer offset
>   with EnclosingIntRect() when the display items are added into
>   WebDisplayItemList.
>
> Please see the bug for the reason.
>
> BUG=724263
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
>
> Review-Url: https://codereview.chromium.org/2894093002
> Cr-Commit-Position: refs/heads/master@{#473371}
> Committed: https://chromium.googlesource.com/chromium/src/+/3c7f1c4ecd1421d074daa5a103ea3f895c1c8724

TBR=chrishtr@chromium.org,wkorman@chromium.org,pdr@chromium.org,michaelpg@chromium.org
BUG=724263
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2899793002
Cr-Original-Commit-Position: refs/heads/master@{#473640}
Review-Url: https://codereview.chromium.org/2897943004
Cr-Commit-Position: refs/branch-heads/3107@{#4}
Cr-Branched-From: 0461d19b57461980dde77a7f4f5e5c3c80d4dc4b-refs/heads/master@{#473482}

[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/core/frame/FrameView.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/core/paint/PaintControllerPaintTest.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/BUILD.gn
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/ContentLayerDelegate.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/ClipDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/ClipDisplayItem.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/ClipPathDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/ClipPathDisplayItem.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/CompositingDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/CompositingDisplayItem.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/DisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/DisplayItem.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/DisplayItemClient.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/DisplayItemList.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/DisplayItemList.h
[add] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/DisplayItemListTest.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/DrawingDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/DrawingDisplayItem.h
[delete] https://crrev.com/700a4414d338475949c04337cad6aec760d2c53d/third_party/WebKit/Source/platform/graphics/paint/DrawingDisplayItemTest.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/FilterDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/FilterDisplayItem.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/FloatClipDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/FloatClipDisplayItem.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/ForeignLayerDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/ForeignLayerDisplayItem.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/PaintArtifact.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/PaintArtifact.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/PaintChunkerTest.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/PaintController.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/PaintController.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/ScrollDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/ScrollDisplayItem.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/Transform3DDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/Transform3DDisplayItem.h
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/TransformDisplayItem.cpp
[modify] https://crrev.com/238659de163326395f9628bfc9cac04b55678f88/third_party/WebKit/Source/platform/graphics/paint/TransformDisplayItem.h

Comment 11 by sandeepkumars@chromium.org, May 23 2017

Labels: TE-Verified-60.0.3107.4 TE-Verified-60
Verified this issue on Win 10, Mac 10.12.4, Ubuntu 14.04 using chrome latest dev M60 #60.0.3107.4 by following steps mentioned in the original comment. 

No delay is observed on opening the sub page. Hence adding the TE- Verified label

Please refer the screen cast

Thanks,
May 23 2017 1-53 PM.webm
2.9 MB View Download

Comment 12 by wangxianzhu@chromium.org, May 23 2017

Status: Fixed (was: Assigned)

Comment 13 by wangxianzhu@chromium.org, May 23 2017

Blocking: 724263

Comment 14 by abod...@chromium.org, Jun 15 2017

Status: Verified (was: Fixed)
►

Sign in to add a comment