components/arc/DEPS - Issue 1828533002: Enable UMA histogram logger for low memor kill events in ArcMetricsService.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: components/arc/DEPS

Issue 1828533002: Enable UMA histogram logger for low memor kill events in ArcMetricsService. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Fix NaClBrowserTest Created 4 years, 9 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« chrome/test/nacl/nacl_browsertest.cc ('K') | « components/arc/BUILD.gn ('k') | components/arc/metrics/arc_low_memory_killer_monitor.h » ('j') | components/arc/metrics/arc_low_memory_killer_monitor.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 include_rules = [	1 include_rules = [

2 "+chromeos/arc",	2 "+chromeos/arc",

3 "+chromeos/chromeos_switches.h",	3 "+chromeos/chromeos_switches.h",

4 "+chromeos/dbus",	4 "+chromeos/dbus",

5 "+components/exo",	5 "+components/exo",

6 "+components/prefs",	6 "+components/prefs",

7 "+components/signin/core/account_id",	7 "+components/signin/core/account_id",

8 "+ipc",	8 "+ipc",

9 "+mojo",	9 "+mojo",

	10 "+third_party/re2",
	tfarina 2016/03/27 08:03:59 You are adding now but for a line that is already You are adding now but for a line that is already exist, I feel this should be done separately from this change. But since it is so small, I'm ok with it being part of this CL. Just wanted to note that. cylee1 2016/03/28 05:24:23 As explained by hidehiko, this file is moved from Show quoted text On 2016/03/27 08:03:59, tfarina wrote: > You are adding now but for a line that is already exist, I feel this > should be done separately from this change. > > But since it is so small, I'm ok with it being part of this CL. Just wanted to > note that. As explained by hidehiko, this file is moved from other location. That's why this line is added in this CL.
10 "+ui/arc",	11 "+ui/arc",

11 ]	12 ]

OLD	NEW