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

Issue 123613005: [Mac] Resolve trybot flakiness: DCHECK fail when deleting GL shader programs. (Closed)

Created:
6 years, 11 months ago by miu
Modified:
6 years, 11 months ago
Reviewers:
Alpha Left Google
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, miu+watch_chromium.org
Visibility:
Public.

Description

[Mac] Resolve trybot flakiness: DCHECK fail when deleting GL shader programs. The code has other protections to ensure all programs will be eventually deleted, and only once for each time they're created. BUG=326328 R=hclam@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=243196

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -2 lines) Patch
M content/browser/renderer_host/compositing_iosurface_shader_programs_mac.cc View 1 chunk +0 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
miu
Alpha: PTAL. I believe the flakiness is either due to a GL driver bug or ...
6 years, 11 months ago (2014-01-06 21:14:05 UTC) #1
Alpha Left Google
LGTM.
6 years, 11 months ago (2014-01-06 21:15:26 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/miu@chromium.org/123613005/1
6 years, 11 months ago (2014-01-06 21:29:37 UTC) #3
miu
All Mac trybots passed. Directly committing since this flake is happening too often, slowing down ...
6 years, 11 months ago (2014-01-06 23:14:20 UTC) #4
miu
6 years, 11 months ago (2014-01-06 23:16:10 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 manually as r243196 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698