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

Issue 284193006: Revert 270894 "Use real PlatformSupport in content unit tests" (Closed)

Created:
6 years, 7 months ago by benwells
Modified:
6 years, 7 months ago
Reviewers:
enne (OOO)
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 270894 "Use real PlatformSupport in content unit tests" > Use real PlatformSupport in content unit tests > > Once all pages are force composited by default, this allows these tests > to not explode when trying to create GraphicsLayers because of a lack of > Platform::compositorSupport. > > BUG=none > > Review URL: https://codereview.chromium.org/287043002 This test has caused leak errors on the mac valgrind bot: http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20Mac%20%28valgrind%29%282%29/builds/27964 TBR=enne@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270950

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+40 lines, -18 lines) Patch
M trunk/src/content/content_tests.gypi View 5 chunks +12 lines, -12 lines 0 comments Download
M trunk/src/content/public/test/unittest_test_suite.h View 2 chunks +2 lines, -2 lines 0 comments Download
M trunk/src/content/public/test/unittest_test_suite.cc View 1 chunk +26 lines, -4 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
benwells
6 years, 7 months ago (2014-05-16 06:26:22 UTC) #1
benwells
6 years, 7 months ago (2014-05-16 06:26:41 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r270950.

Powered by Google App Engine
This is Rietveld 408576698