Issue 697050 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	bcwh...@chromium.org
Closed:	Mar 2017
Cc:	█ manzagop@chromium.org
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	3
Type:	Bug

Blocking:
issue 620813

ActivityUserData doesn't record empty strings.

Project Member Reported by manzagop@chromium.org, Feb 28 2017

Issue description

I noticed empty strings don't get recorded with the current logic.
https://cs.chromium.org/chromium/src/base/debug/activity_tracker.cc?l=335

Project Member

Comment 1 by bugdroid1@chromium.org, Mar 7 2017

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

commit 1c36e4ed00bdc18fa8c3892f40e1a801f32a4d54
Author: bcwhite <bcwhite@chromium.org>
Date: Tue Mar 07 23:30:41 2017

Support zero-sized values.

BUG= 697050 

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

[modify] https://crrev.com/1c36e4ed00bdc18fa8c3892f40e1a801f32a4d54/base/debug/activity_tracker.cc
[modify] https://crrev.com/1c36e4ed00bdc18fa8c3892f40e1a801f32a4d54/base/debug/activity_tracker_unittest.cc

Comment 2 by bcwh...@chromium.org, Mar 14 2017

Status: Fixed (was: Assigned)

►

Issue 697050 link

Issue metadata

ActivityUserData doesn't record empty strings.

Issue description

Comment 1 by bugdroid1@chromium.org, Mar 7 2017

Comment 1 Deleted

Comment 2 by bcwh...@chromium.org, Mar 14 2017

Comment 2 Deleted

Sign in to add a comment