Revert of Do not produce dSYM file for the iOS Frameworks with bitcode

Revert of Do not produce dSYM file for the iOS Frameworks with bitcode (patchset #2 id:20001 of https://codereview.webrtc.org/2705163007/ )

Reason for revert:
Looks like this caused the iOS API Framework Builder to fail.

https://build.chromium.org/p/client.webrtc/builders/iOS%20API%20Framework%20Builder/builds/3487/steps/zip%20archive/logs/stdio

Zipping /b/rr/tmpkIyP1e/w/webrtc_ios_api_framework.zip...
Traceback (most recent call last):
  File "/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py", line 144, in <module>
    sys.exit(main())
  File "/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py", line 130, in main
    exit_code = zip_with_subprocess(root, output, entries)
  File "/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py", line 43, in zip_with_subprocess
    assert os.path.isdir(path), path
AssertionError: /b/c/b/iOS_API_Framework_Builder/src/out_ios_libs/WebRTC.dSYM/
step returned non-zero exit code: 1
@@@STEP_FAILURE@@@

Original issue's description:
> Do not produce dSYM file for the iOS Frameworks with bitcode
> 
> Though dSYM files can be generated when building applications or libraries
> with bitcode. They cannot be used to symbolicate crash reports from
> applications. Instead, developers need to grab the real dSYM files, which
> are generated for each specific device type after uploading an iOS / tvOS
> application to App Store (or to a device using Xcode). Apple clearly warns
> about it in its documentation:
> 
> https://developer.apple.com/library/content/technotes/tn2151/_index.html#//apple_ref/doc/uid/DTS40008184-CH1-SYMBOLICATION-BITCODE
> 
> With that in mind, I believe that it would be better to not confuse
> developers by giving them dSYM files that are not very helpful with
> the bitcode-enabled framework. Thus, proposing the following modification
> to the building script, to generate dSYM by default only without
> the bitcode option. However, if some developers still want to get
> the dSYM files as a build-process artifact, when enabling bitcode,
> they can explicitly add --extra-gn-args enable_dsyms=true to the script.
> 
> Let me know if it lgty.
> 
> NOTRY=True
> BUG=None
> 
> Review-Url: https://codereview.webrtc.org/2705163007
> Cr-Commit-Position: refs/heads/master@{#16836}
> Committed: https://chromium.googlesource.com/external/webrtc/+/d74517c52a6cd4172b1f3fdc4e624b6145ff5a0f

TBR=kjellander@webrtc.org,kthelgason@webrtc.org,VladimirTechMan@gmail.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=None
NOTRY=True

Review-Url: https://codereview.webrtc.org/2719773002
Cr-Commit-Position: refs/heads/master@{#16844}
Committed: https://chromium.googlesource.com/external/webrtc/+/da00077cfac335503b3335d5eac110f2f5bff408

[tommi, 2017-02-26 14:07:16]

The CQ bit was checked by tommi@webrtc.org

[tommi, 2017-02-26 14:07:16]

Created Revert of Do not produce dSYM file for the iOS Frameworks with bitcode

[commit-bot: I haz the power, 2017-02-26 14:07:19]

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

[tommi, 2017-02-26 14:36:19]

Description was changed from

==========
Revert of Do not produce dSYM file for the iOS Frameworks with bitcode (patchset
#2 id:20001 of https://codereview.webrtc.org/2705163007/ )

Reason for revert:
Looks like this caused the iOS API Framework Builder to fail.

https://build.chromium.org/p/client.webrtc/builders/iOS%20API%20Framework%20B...

Zipping /b/rr/tmpkIyP1e/w/webrtc_ios_api_framework.zip...
Traceback (most recent call last):
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 144, in <module>
    sys.exit(main())
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 130, in main
    exit_code = zip_with_subprocess(root, output, entries)
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 43, in zip_with_subprocess
    assert os.path.isdir(path), path
AssertionError: /b/c/b/iOS_API_Framework_Builder/src/out_ios_libs/WebRTC.dSYM/
step returned non-zero exit code: 1
@@@STEP_FAILURE@@@

Original issue's description:
> Do not produce dSYM file for the iOS Frameworks with bitcode
> 
> Though dSYM files can be generated when building applications or libraries
> with bitcode. They cannot be used to symbolicate crash reports from
> applications. Instead, developers need to grab the real dSYM files, which
> are generated for each specific device type after uploading an iOS / tvOS
> application to App Store (or to a device using Xcode). Apple clearly warns
> about it in its documentation:
> 
>
https://developer.apple.com/library/content/technotes/tn2151/_index.html#//ap...
> 
> With that in mind, I believe that it would be better to not confuse
> developers by giving them dSYM files that are not very helpful with
> the bitcode-enabled framework. Thus, proposing the following modification
> to the building script, to generate dSYM by default only without
> the bitcode option. However, if some developers still want to get
> the dSYM files as a build-process artifact, when enabling bitcode,
> they can explicitly add --extra-gn-args enable_dsyms=true to the script.
> 
> Let me know if it lgty.
> 
> NOTRY=True
> BUG=None
> 
> Review-Url: https://codereview.webrtc.org/2705163007
> Cr-Commit-Position: refs/heads/master@{#16836}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/d74517c52a6cd4172b1f3fdc4...

TBR=kjellander@webrtc.org,kthelgason@webrtc.org,VladimirTechMan@gmail.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=None
==========

to

==========
Revert of Do not produce dSYM file for the iOS Frameworks with bitcode (patchset
#2 id:20001 of https://codereview.webrtc.org/2705163007/ )

Reason for revert:
Looks like this caused the iOS API Framework Builder to fail.

https://build.chromium.org/p/client.webrtc/builders/iOS%20API%20Framework%20B...

Zipping /b/rr/tmpkIyP1e/w/webrtc_ios_api_framework.zip...
Traceback (most recent call last):
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 144, in <module>
    sys.exit(main())
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 130, in main
    exit_code = zip_with_subprocess(root, output, entries)
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 43, in zip_with_subprocess
    assert os.path.isdir(path), path
AssertionError: /b/c/b/iOS_API_Framework_Builder/src/out_ios_libs/WebRTC.dSYM/
step returned non-zero exit code: 1
@@@STEP_FAILURE@@@

Original issue's description:
> Do not produce dSYM file for the iOS Frameworks with bitcode
> 
> Though dSYM files can be generated when building applications or libraries
> with bitcode. They cannot be used to symbolicate crash reports from
> applications. Instead, developers need to grab the real dSYM files, which
> are generated for each specific device type after uploading an iOS / tvOS
> application to App Store (or to a device using Xcode). Apple clearly warns
> about it in its documentation:
> 
>
https://developer.apple.com/library/content/technotes/tn2151/_index.html#//ap...
> 
> With that in mind, I believe that it would be better to not confuse
> developers by giving them dSYM files that are not very helpful with
> the bitcode-enabled framework. Thus, proposing the following modification
> to the building script, to generate dSYM by default only without
> the bitcode option. However, if some developers still want to get
> the dSYM files as a build-process artifact, when enabling bitcode,
> they can explicitly add --extra-gn-args enable_dsyms=true to the script.
> 
> Let me know if it lgty.
> 
> NOTRY=True
> BUG=None
> 
> Review-Url: https://codereview.webrtc.org/2705163007
> Cr-Commit-Position: refs/heads/master@{#16836}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/d74517c52a6cd4172b1f3fdc4...

TBR=kjellander@webrtc.org,kthelgason@webrtc.org,VladimirTechMan@gmail.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=None
NOTRY=True
==========

[tommi, 2017-02-26 14:36:19]

tommi@webrtc.org changed reviewers:
+ vladimirtechman@gmail.com
- VladimirTechMan@gmail.com

[tommi, 2017-02-26 14:36:25]

The CQ bit was unchecked by tommi@webrtc.org

[tommi, 2017-02-26 14:36:38]

The CQ bit was checked by tommi@webrtc.org

[commit-bot: I haz the power, 2017-02-26 14:36:43]

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, 2017-02-26 14:39:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488119798425440, "parent_rev":
"922246a353acff8445d4b104f0271befff2bfa68", "commit_rev":
"da00077cfac335503b3335d5eac110f2f5bff408"}

[commit-bot: I haz the power, 2017-02-26 14:39:11]

Description was changed from

==========
Revert of Do not produce dSYM file for the iOS Frameworks with bitcode (patchset
#2 id:20001 of https://codereview.webrtc.org/2705163007/ )

Reason for revert:
Looks like this caused the iOS API Framework Builder to fail.

https://build.chromium.org/p/client.webrtc/builders/iOS%20API%20Framework%20B...

Zipping /b/rr/tmpkIyP1e/w/webrtc_ios_api_framework.zip...
Traceback (most recent call last):
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 144, in <module>
    sys.exit(main())
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 130, in main
    exit_code = zip_with_subprocess(root, output, entries)
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 43, in zip_with_subprocess
    assert os.path.isdir(path), path
AssertionError: /b/c/b/iOS_API_Framework_Builder/src/out_ios_libs/WebRTC.dSYM/
step returned non-zero exit code: 1
@@@STEP_FAILURE@@@

Original issue's description:
> Do not produce dSYM file for the iOS Frameworks with bitcode
> 
> Though dSYM files can be generated when building applications or libraries
> with bitcode. They cannot be used to symbolicate crash reports from
> applications. Instead, developers need to grab the real dSYM files, which
> are generated for each specific device type after uploading an iOS / tvOS
> application to App Store (or to a device using Xcode). Apple clearly warns
> about it in its documentation:
> 
>
https://developer.apple.com/library/content/technotes/tn2151/_index.html#//ap...
> 
> With that in mind, I believe that it would be better to not confuse
> developers by giving them dSYM files that are not very helpful with
> the bitcode-enabled framework. Thus, proposing the following modification
> to the building script, to generate dSYM by default only without
> the bitcode option. However, if some developers still want to get
> the dSYM files as a build-process artifact, when enabling bitcode,
> they can explicitly add --extra-gn-args enable_dsyms=true to the script.
> 
> Let me know if it lgty.
> 
> NOTRY=True
> BUG=None
> 
> Review-Url: https://codereview.webrtc.org/2705163007
> Cr-Commit-Position: refs/heads/master@{#16836}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/d74517c52a6cd4172b1f3fdc4...

TBR=kjellander@webrtc.org,kthelgason@webrtc.org,VladimirTechMan@gmail.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=None
NOTRY=True
==========

to

==========
Revert of Do not produce dSYM file for the iOS Frameworks with bitcode (patchset
#2 id:20001 of https://codereview.webrtc.org/2705163007/ )

Reason for revert:
Looks like this caused the iOS API Framework Builder to fail.

https://build.chromium.org/p/client.webrtc/builders/iOS%20API%20Framework%20B...

Zipping /b/rr/tmpkIyP1e/w/webrtc_ios_api_framework.zip...
Traceback (most recent call last):
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 144, in <module>
    sys.exit(main())
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 130, in main
    exit_code = zip_with_subprocess(root, output, entries)
  File
"/b/rr/tmpkIyP1e/rw/checkout/scripts/slave/recipe_modules/zip/resources/zip.py",
line 43, in zip_with_subprocess
    assert os.path.isdir(path), path
AssertionError: /b/c/b/iOS_API_Framework_Builder/src/out_ios_libs/WebRTC.dSYM/
step returned non-zero exit code: 1
@@@STEP_FAILURE@@@

Original issue's description:
> Do not produce dSYM file for the iOS Frameworks with bitcode
> 
> Though dSYM files can be generated when building applications or libraries
> with bitcode. They cannot be used to symbolicate crash reports from
> applications. Instead, developers need to grab the real dSYM files, which
> are generated for each specific device type after uploading an iOS / tvOS
> application to App Store (or to a device using Xcode). Apple clearly warns
> about it in its documentation:
> 
>
https://developer.apple.com/library/content/technotes/tn2151/_index.html#//ap...
> 
> With that in mind, I believe that it would be better to not confuse
> developers by giving them dSYM files that are not very helpful with
> the bitcode-enabled framework. Thus, proposing the following modification
> to the building script, to generate dSYM by default only without
> the bitcode option. However, if some developers still want to get
> the dSYM files as a build-process artifact, when enabling bitcode,
> they can explicitly add --extra-gn-args enable_dsyms=true to the script.
> 
> Let me know if it lgty.
> 
> NOTRY=True
> BUG=None
> 
> Review-Url: https://codereview.webrtc.org/2705163007
> Cr-Commit-Position: refs/heads/master@{#16836}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/d74517c52a6cd4172b1f3fdc4...

TBR=kjellander@webrtc.org,kthelgason@webrtc.org,VladimirTechMan@gmail.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=None
NOTRY=True

Review-Url: https://codereview.webrtc.org/2719773002
Cr-Commit-Position: refs/heads/master@{#16844}
Committed:
https://chromium.googlesource.com/external/webrtc/+/da00077cfac335503b3335d5e...
==========

[commit-bot: I haz the power, 2017-02-26 14:39:12]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/da00077cfac335503b3335d5e...

[kthelgason, 2017-02-27 07:30:14]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2718983002/ by kthelgason@webrtc.org.

The reason for reverting is: Fixing issues with the framework builder..