Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1885)

Issue 176633003: Disable another composited query assertion. (Closed)

Created:
6 years, 10 months ago by dcheng
Modified:
6 years, 10 months ago
Reviewers:
abarth-chromium, ojan
CC:
blink-reviews, bemjb+rendering_chromium.org, dsinclair, zoltan1, eae+blinkwatch, leviw+renderwatch, blink-layers+watch_chromium.org, jchaffraix+rendering, pdr., ojan
Visibility:
Public.

Description

Disable another composited query assertion. Hopefully this is the last of them... TBR=abarth@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=167648

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M Source/core/rendering/RenderLayerScrollableArea.cpp View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
dcheng
Committed patchset #1 manually as r167648 (tree was closed).
6 years, 10 months ago (2014-02-23 18:57:29 UTC) #1
abarth-chromium
lgtm Thanks!
6 years, 10 months ago (2014-02-23 19:04:11 UTC) #2
ojan
Which test needed this? We need to add that in a comment so that the ...
6 years, 10 months ago (2014-02-24 18:06:26 UTC) #3
dcheng
6 years, 10 months ago (2014-02-24 18:28:13 UTC) #4
Message was sent while issue was closed.
On 2014/02/24 18:06:26, ojan wrote:
> Which test needed this? We need to add that in a comment so that the person
who
> goes in to fix this issue knows what test to run.

Building with GYP_DEFINES="chromeos=1 use_aura=1":
browser_tests
--gtest_filter=KioskHiddenWebUITest.AutolaunchWarning:OptionsWebUIExtendedTest.OverlayTabNavigation:PrintPreviewWebUITest.PrintScalingDisabledForPlugin:PrintPreviewWebUITest.PrintToPDFSelectedHideFitToPageOption:PrintPreviewWebUITest.SourceIsPDFShowFitToPageOption:RestorePrefs/FileManagerBrowserTest.Test/2:RestorePrefs/FileManagerBrowserTest.Test/3:TermsOfServiceTestInstance/TermsOfServiceTest.TermsOfServiceScreen/0

I didn't want to add references to Chrome tests in Blink, but if there's
precedence for that....
The other suppression I added (https://codereview.chromium.org/176633003/) was
triggered only the PrintPreviewWebUITests listed above.

Powered by Google App Engine
This is Rietveld 408576698