[GN] Add missing framework headers

[GN] Add missing framework headers

This CL adds headers that are present in the /Headers directory but not included in the framework target.

BUG=none
NOTRY=true

Committed: https://crrev.com/8fea1992f8b4c0f1c57f3ebae6f44f0f197551a4
Cr-Commit-Position: refs/heads/master@{#14413}

[kthelgason, 2016-09-16 07:39:49]

Description was changed from

==========
[GN] Add missing framework headers

BUG=none
==========

to

==========
[GN] Add missing framework headers

This CL adds headers that are present in the /Headers directory but not included
in the framework target.

BUG=none
NOTRY=true
==========

[kthelgason, 2016-09-16 07:39:49]

kthelgason@webrtc.org changed reviewers:
+ kjellander@webrtc.org, tkchin@webrtc.org

[kjellander_webrtc, 2016-09-18 17:45:38]

https://codereview.webrtc.org/2342293002/diff/1/webrtc/sdk/BUILD.gn
File webrtc/sdk/BUILD.gn (right):

https://codereview.webrtc.org/2342293002/diff/1/webrtc/sdk/BUILD.gn#newcode233
webrtc/sdk/BUILD.gn:233: "objc/Framework/Headers/WebRTC/RTCLogging.h",
Since objc/Framework/Headers/WebRTC/RTCFileLogger.h is handled differently below
(even if this target is not even referenced from Chromium, I'm wondering if this
should go into that case as well). I guess we'll have to wait for Zeke's
comments.

[tkchin_webrtc, 2016-09-27 09:39:59]

lgtm

https://codereview.webrtc.org/2342293002/diff/1/webrtc/sdk/BUILD.gn
File webrtc/sdk/BUILD.gn (right):

https://codereview.webrtc.org/2342293002/diff/1/webrtc/sdk/BUILD.gn#newcode233
webrtc/sdk/BUILD.gn:233: "objc/Framework/Headers/WebRTC/RTCLogging.h",
On 2016/09/18 17:45:38, kjellander_webrtc wrote:
> Since objc/Framework/Headers/WebRTC/RTCFileLogger.h is handled differently
below
> (even if this target is not even referenced from Chromium, I'm wondering if
this
> should go into that case as well). I guess we'll have to wait for Zeke's
> comments.

https://codereview.webrtc.org/1898283002/ for context
It's okay to leave RTCLogging.h

[kthelgason, 2016-09-28 09:01:48]

The CQ bit was checked by kthelgason@webrtc.org

[commit-bot: I haz the power, 2016-09-28 09:01:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-28 09:06:56]

Description was changed from

==========
[GN] Add missing framework headers

This CL adds headers that are present in the /Headers directory but not included
in the framework target.

BUG=none
NOTRY=true
==========

to

==========
[GN] Add missing framework headers

This CL adds headers that are present in the /Headers directory but not included
in the framework target.

BUG=none
NOTRY=true
==========

[commit-bot: I haz the power, 2016-09-28 09:06:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-28 09:07:06]

Description was changed from

==========
[GN] Add missing framework headers

This CL adds headers that are present in the /Headers directory but not included
in the framework target.

BUG=none
NOTRY=true
==========

to

==========
[GN] Add missing framework headers

This CL adds headers that are present in the /Headers directory but not included
in the framework target.

BUG=none
NOTRY=true

Committed: https://crrev.com/8fea1992f8b4c0f1c57f3ebae6f44f0f197551a4
Cr-Commit-Position: refs/heads/master@{#14413}
==========

[commit-bot: I haz the power, 2016-09-28 09:07:07]

Patchset 1 (id:??) landed as
https://crrev.com/8fea1992f8b4c0f1c57f3ebae6f44f0f197551a4
Cr-Commit-Position: refs/heads/master@{#14413}