blimp/BUILD.gn - Issue 1324263003: Blimp: create MessageDispatcher class.

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: blimp/BUILD.gn

Issue 1324263003: Blimp: create MessageDispatcher class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@blimp-protos

Patch Set: Remove testonly annotation Created 5 years, 3 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:

OLD	NEW
1 # Copyright 2015 The Chromium Authors. All rights reserved.	1 # Copyright 2015 The Chromium Authors. All rights reserved.

2 # Use of this source code is governed by a BSD-style license that can be	2 # Use of this source code is governed by a BSD-style license that can be

3 # found in the LICENSE file.	3 # found in the LICENSE file.

4	4

	5 # Metatarget for all of Blimp's feature code.

5 group("blimp") {	6 group("blimp") {

6 deps = [	7 deps = [

7 "//blimp/client:blimp_client",	8 "//blimp/client:blimp_client",

8 "//blimp/common:blimp_common",	9 "//blimp/common:blimp_common",

9 ]	10 ]

10 }	11 }

	12

	13 # Metatarget for all of Blimp's unit tests.

	14 group("blimp_tests") {

	15 testonly = true

	16 deps = [

	17 "common:blimp_common_unit_tests",
	nyquist 2015/09/10 20:12:09 I see now that the style of this is different than I see now that the style of this is different than in the group above. Should we be consistent with //blimp/... prefix for everything that's not local? Kevin M 2015/09/23 00:05:30 Done. Show quoted text On 2015/09/10 20:12:09, nyquist wrote: > I see now that the style of this is different than in the group above. Should we > be consistent with //blimp/... prefix for everything that's not local? Done.
	18 ]

	19 }

OLD	NEW