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

Issue 2843433003: cc: Remove slow BeginMainFrame dump. (Closed)

Created:
3 years, 8 months ago by sunnyps
Modified:
3 years, 8 months ago
Reviewers:
vmpstr
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3071
Project:
chromium
Visibility:
Public.

Description

cc: Remove slow BeginMainFrame dump. This is for M59 branch only. We'd like to keep the dump for canary but we need to land it on trunk, ship in one canary, merge to branch, and then revert on trunk. R=vmpstr BUG=668892, 622080, 711064 Change-Id: I78031639da8ac09925e080b6af342e075909bafb CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel Change-Id: I78031639da8ac09925e080b6af342e075909bafb Reviewed-on: https://chromium-review.googlesource.com/484703 Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org>; Reviewed-by: Vladimir Levin <vmpstr@chromium.org>; Cr-Original-Commit-Position: refs/heads/master@{#466444}(cherry picked from commit 40f1892ad93b2dc497141e351fddf32475b8cd62) Review-Url: https://codereview.chromium.org/2843433003 . Cr-Commit-Position: refs/branch-heads/3071@{#171} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} Committed: https://chromium.googlesource.com/chromium/src/+/8ee798628e7bc80a2f60da06b710f251ea89ee3e

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -11 lines) Patch
M cc/trees/proxy_main.cc View 1 chunk +1 line, -11 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
sunnyps
3 years, 8 months ago (2017-04-24 18:35:20 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
8ee798628e7bc80a2f60da06b710f251ea89ee3e.

Powered by Google App Engine
This is Rietveld 408576698