Fix link typo in gn cookbook doc.

tools/gn/docs/cookbook.md

 # GYP->GN Conversion Cookbook
 
 [TOC]
 
 ## Targets
 
 | *GYP*                                            | *GN*                                               |
 |:-------------------------------------------------|:---------------------------------------------------|
 | `'type': 'static_library', 'name': 'foo',`       | `static_library("foo") {` or `source_set("foo") {` |
 | `'type': 'shared_library', 'name': 'foo',`       | `shared_library("foo") {`                          |
 | `'type': '<(component)', 'name': 'foo',`         | `component("foo") {`                               |
 | `'type': 'executable', 'name': 'foo',`           | `executable("foo") {`                              |
 | `'type': '<(gtest_target_type)', 'name': 'foo',` | `test("foo") {`                                    |
 | `'type': 'none', 'name': 'foo',`                 | `group("foo") {`                                   |
 
 ### Note on static libraries
 
 A source\_set is basically a transparent static\_library. The source files
 are compiled with the given options but not linked into anything.
 Targets that depend on a source set get the source set's object files
 linked into it. This saves the overhead of creating a static library on
 disk, avoids weird linker issues when a static library has no source
 files, and you can link source sets into shared libraries and have
 symbols exported from the shared library.
 
 The last issue is a cause of a lot of headaches in the GYP build. If you
 annotate a symbol as exported (i.e. `BASE_EXPORT`) then you can't have
 it in a file that goes into a static library because the function might
-be [http://blogs.msdn.com/b/oldnewthing/archive/2014/03/21/10509670.aspx
-stripped out] if it's not called from within the static library. This

[Dirk Pranke, 2015/08/03 19:18:09]

It actually needs to be [text](link), not (text)[link].

[Bons, 2015/08/03 19:23:00]

Done.

+be (stripped out)[http://blogs.msdn.com/b/oldnewthing/archive/2014/03/21/10509670.aspx]
+if it's not called from within the static library. This
 prevents composing components of static libraries and exporting their
 symbols. A source set avoids this issue and `EXPORT` has the desired
 meaning of "export from the component this gets linked into" with no
 surprising dead code stripping behavior.
 
 A disadvantage of source sets is that if an object file is completely
 unused, it will still be linked into the result, which is not the case
 for static libraries. A few places in the build depend on this behavior
 (deliberately or accidentally). In general, small libraries that we
 expect to be entirely used, test helpers, etc. can be source sets. There
@@ skipping 65 matching lines @@
     ],
     'dependencies': [
       '<(DEPTH)/base/base.gyp:bar',
     ],
   }, {
     'sources/': [
       ['exclude', '^b\\.cc$'],
     ],
   }],
 ],
-```  
+```
 
-### GN 
+### GN
 
 ```
 sources = [
   "a.cc",
   "b.cc",
 ]
 deps = [
   "//base:foo",
 ]
 
@@ skipping 31 matching lines @@
 declared directly in that file (so other files can't use them). These
 places are noted in the table below.
 
 | *GYP*                                           | *GN*                                       | *GN import*                                    |
 |:------------------------------------------------|:-------------------------------------------|:-----------------------------------------------|
 | `arm_float_abi`                                 | `arm_float_abi`                            | `//build/config/arm.gni`                       |
 | `arm_neon` (0/1)                                | `arm_use_neon` (true/false)                | `//build/config/arm.gni`                       |
 | `arm_neon_optional` (0/1)                       | `arm_optionally_use_neon` (true/false)     | `//build/config/arm.gni`                       |
 | `arm_version`                                   | `arm_version`                              | `//build/config/arm.gni`                       |
 | `asan` (0/1)                                    | `is_asan` (true/false)                     | (global)                                       |
 | `branding` ("Chromium"/"Chrome"")               | `is_chrome_branded` (true/false)           | `//build/config/chrome_build.gni`              |

[Dirk Pranke, 2015/08/03 19:18:09]

There's an extra double quote here (after the "Chrome"), which is I think what's
messing up the rest of the table. Can you fix this while you're at it?

[Bons, 2015/08/03 19:23:00]

Done.

 | `build_for_tool=="drmemory"`                    | `disable_iterator_debugging` (true/false)  | (internal to `//build/config/BUILD.gn`)        |
 | `build_for_tool=="tsan"`                        | `disable_iterator_debugging` (true/false)  | (internal to `//build/config/BUILD.gn`)        |
 | `buildtype` ("Official"/"Dev")                  | `is_official_build` (true/false)           | `//build/config/chrome_build.gni`              |
 | `chrome_multiple_dll` (0/1)                     | `is_multi_dll_chrome` (true/false)         | `//build/config/chrome_build.gni`              |
 | `clang` (0/1)                                   | `is_clang` (true/false)                    | (global)                                       |
 | `clang_use_chrome_plugins` (0/1)                | `clang_use_chrome_plugins` (true/false)    | (internal to `//build/config/clang/BUILD.gn`)  |
 | `component` ("shared_library"/"static_library") | `is_component_build` (true/false)          | (global)                                       |
 | `desktop_linux` (0/1)                           | `is_desktop_linux` (true/false)            | (global)                                       |
 | `disable_glibcxx_debug` (0/1)                   | `disable_iterator_debugging` (true/false)  | (internal to `//build/config/BUILD.gn`)        |
 | `fastbuild` (0/1/2)                             | `symbol_level` (2/1/0 — values inverted)   | (global)                                       |
 | `gomadir`                                       | `goma_dir`                                 | `//build/toolchain/goma.gni`                   |
 | `ios_deployment_target` (string)                | `ios_deployment_target`                    | `//build/config/ios/ios_sdk.gni`               |
 | `GYP_MSVS_OVERRIDE_PATH` environment variable   | `visual_studio_path`                       | `//build/config/win/visual_studio_version.gni` |
 | `GYP_MSVS_VERSION` environment variable         | (none)                                     |                                                |
 | `ios_sdk_path`                                  | `ios_sdk_path` and `use_ios_simulator`     | `//build/config/ios/ios_sdk.gni`               |
 | `lsan` (0/1)                                    | `is_lsan` (true/false)                     | (global)                                       |
 | `mac_sdk_min`                                   | `mac_sdk_min`                              | `//build/config/mac/mac_sdk.gni`               |
 | `mac_sdk_path`                                  | `mac_sdk_path`                             | `//build/config/mac/mac_sdk.gni`               |
 | `mac_sdk`                                       | `mac_sdk_version`                          | `//build/config/mac/mac_sdk.gni`               |
 | `msan` (0/1)                                    | `is_msan` (true/false)                     | (global)                                       |
 | `SDKROOT` (Mac)                                 | `sysroot`                                  | `//build/config/sysroot.gni`                   |
 | `sysroot`                                       | `sysroot`                                  | `//build/config/sysroot.gni`                   |
 | `target_arch` ("ia32"/"x64"/"arm"/"mipsel")     | `target_arch` ("x86"/"x64"/"arm"/"mipsel") | (global)                                       |
 | `toolkit_views` (0/1)                           | `toolkit_views`                            | `//build/config/ui.gni`                        |
 | `tsan` (0/1)                                    | `is_tsan` (true/false)                     | (global)                                       |
 | `windows_sdk_path`                              | `windows_sdk_path`                         | (internal to `//build/config/win/BUILD.gn`)    |
 
-### Feature flags 
+### Feature flags
 
 | *GYP*                                   | *GN*                                           | *GN import*                   |
 |:----------------------------------------|:-----------------------------------------------|:------------------------------|
 | `cld_version` (number)                  | `cld_version` (number)                         | `//build/config/features.gni` |
 | `configuration_policy` (0/1)            | `enable_configuration_policy` (true/false)     | `//build/config/features.gni` |
 | `debug_devtools` (0/1)                  | `debug_devtools` (true/false)                  | `//build/config/features.gni` |
 | `disable_ftp_support` (0/1)             | `disable_ftp_support` (true/false)             | `//build/config/features.gni` |
 | `disable_nacl` (0/1)                    | `enable_nacl` (true/false)                     | `//build/config/features.gni` |
 | `enable_app_list` (0/1)                 | `enable_app_list` (true/false)                 | `//build/config/features.gni` |
 | `enable_autofill_dialog` (0/1)          | `enable_autofill_dialog` (true/false)          | `//build/config/features.gni` |
@@ skipping 44 matching lines @@
 | `use_openssl` (0/1)                     | `use_openssl` (true/false)                     | `//build/config/crypto.gni` (Many of these conditions can be deleted, see the "SSL" notes on targets below.) |
 | `use_pango` (0/1)                       | `use_pango` (true/false)                       | `//build/config/ui.gni`       |
 | `use_ozone` (0/1)                       | `use_ozone` (true/false)                       | `//build/config/ui.gni`       |
 | `use_seccomp_bpf` (0/1)                 | `use_seccomp_bpf` (true/false)                 | `//build/config/features.gni` |
 | `use_udev` (0/1)                        | `use_udev` (true/false)                        | `//build/config/features.gni` |
 | `use_x11` (0/1)                         | `use_x11` (true/false)                         | `//build/config/ui.gni`       |
 | `use_xi2_mt` (0/1)                      | `use_xi2_mt` (true/false)                      | `//build/config/ui.gni`       |
 | `win_pdf_metafile_for_printing` (0/1)   | `win_pdf_metafile_for_printing` (true/false)   | `//build/config/features.gni` |
 | `win_use_allocator_shim` (0/1)          |                                                | (See "Allocator" below)       |
 
-### Common target conversion 
+### Common target conversion
 
 Some targets that lots of projects depend on and how the GN ones
 correspond to GYP ones. (This is for commonly-depended-on or weird
 targets only, don't add stuff here just because you converted it.)
 
 | *GYP*                                                                              | *GN*                                     | *Notes* (see below)  |
 |:-----------------------------------------------------------------------------------|:-----------------------------------------|:---------------------|
 | `base/base.gyp:base`                                                               | `//base`                                 |                      |
 | `base/base.gyp:base_i18n`                                                          | `//base:i18n`                            |                      |
 | `base/base.gyp:run_all_unittests`                                                  | `//base/test:run_all_unittests`          |                      |
@@ skipping 48 matching lines @@
 
 The dependency tree must be a DAG. Some components might share headers
 between a number of internal targets that makes adding the "proper"
 dependencies impossible. In this case, you can separate out a
 `source_set` type target containing just the header(s) in question, and
 make the targets that use that header depend on that source set to break
 the cycle.
 
 ##  Other stuff
 
-### Target conditions 
+### Target conditions
 
 `target_conditions` are like normal conditions but expanded in a
 different phase of GYP. You can generally just convert the conditions
 inside and not worry about the `conditions`/`target_conditions`
 difference.
 
-### xcode_settings 
+### xcode_settings
 
 Some xcode settings are obvious:
 
 ```
   'xcode_settings': {'OTHER_LDFLAGS': ['-foo']},
 ```
 
 Should just expand to:
 
 ```
   ldflags = [ "-foo" ]
 ```
 
 Other flags are less obvious:
 
 ```
   'xcode_settings': { 'GCC_SYMBOLS_PRIVATE_EXTERN': 'NO', },
 ```
 
 These all correspond to various flags that get passed to the compiler or
 linker. You can use your favorite search engine to see what it
 corresponds to, but many of them are not well documented. You can also
 search for the string in
 [https://code.google.com/p/chromium/codesearch#chromium/src/tools/gyp/pylib/gyp/xcode_emulation.py
 tools/gyp/pylib/gyp/xcode_emulation.py]. GYP uses this file to decode

[Dirk Pranke, 2015/08/03 19:18:09]

can you fix this link, too?

[Bons, 2015/08/03 19:23:00]

Done.

 the Xcode settings into command line flags for the ninja build.
 
-### wexit-time destructors 
+### wexit-time destructors
 
 Replace
 
 ```
 'enable_wexit_time_destructors': 1,
 ```
 
 with
 
 ```
 configs += [ "//build/config/compiler:wexit_time_destructors" ]
 ```
 
-### Chromium code 
+### Chromium code
 
 In GYP code is "non-Chromium" by default, and you opt into higher warning levels using:
 
 ```
 'chromium_code': 1,
 ```
 
 In GN, all code is Chromium code by default. If you're compiling a
 static library that needs more lax warnings, opt out of the
 Chromium-code settings with:
 
 ```
 configs -= [ "//build/config/compiler:chromium_code" ]
 configs += [ "//build/config/compiler:no_chromium_code" ]
 ```
 
-### -fvisibility 
+### -fvisibility
 
 All symbols in the build have "hidden" visibility by default (this means
 that symbols aren't exported from shared libraries, a concept different
 than GN's target visibility). If you needed to export all symbols (for a
 third party library) by default in GYP you would do:
 
 ```
 'xcode_settings': [
   'GCC_SYMBOLS_PRIVATE_EXTERN': 'NO',  # no -fvisibility=hidden
 ],
 'cflags!': [
   '-fvisibility=hidden',
 ],
 ```
 
 In GN the equivalent is:
 
 ```
 if (!is_win) {
   configs -= [ "//build/config/gcc:symbol_visibility_hidden" ]
 }
 ```
 
-### Dependent settings 
+### Dependent settings
 
 In GYP you'll see stuff like this, especially in third-party code.
 
 ```
 'direct_dependent_settings': {
   'include_dirs': [
     '.',  # This directory.
     '../..',  # Root "src" path.
   ],
   'defines': [
@@ skipping 26 matching lines @@
 }
 ```
 
 Targets that depend on `foo` will now get `foo_config` applied.
 
 GYP would say `export_dependent_settings` to forward
 `direct_dependent_settings` up the dependency chain. In GN, put the
 dependency in the `public_deps` section and this will happen
 automatically.
 
-### MSVS disabled warnings 
+### MSVS disabled warnings
 
 In GYP you'll see for third-party code:
 
 ```
 'msvs_disabled_warnings': [ 4018, 4244, 4267, ],
 ```
 
 At least half of the warnings in these blocks are already disabled
 globally (we added more global ones later). From the command line, do:
 
@@ skipping 13 matching lines @@
 ```
 if (is_win) {
   cflags += [
     "/wd4267",  # Conversion from size_t to 'type'.
   ]
 }
 ```
 
 (Use `=` instead of `+=` is you haven't already defined a `cflags` variable.)
 
-### Mac frameworks 
+### Mac frameworks
 
 GN knows to convert `.framework` files in the `libs` list to the right
 thing on Mac. You don't need to specify the directories either. So
 convert this:
 
 ```
 'link_settings': {
   'libraries': [
     '$(SDKROOT)/System/Library/Frameworks/Accelerate.framework',
   ],
 },
 ```
 
 to this:
 
 ```
 libs = [ "Accelerate.framework" ]
 ```
 
-### hard_dependency 
+### hard_dependency
 
 GYP code sometimes sets
 
 ```
   'hard_dependency': 1,
 ```
 
 to indicate that the current target must be build before its dependents.
 GN can deduce this internally, so you can ignore this directive.
 
-### Allocator 
+### Allocator
 
 GYP has `win_use_allocator_shim` and `use_allocator`. In GN, these are
 merged into `use_allocator` which is defined in
 `//build/config/allocator.gni`. _However_ you should almost never need
 to use this flag. The `//base/allocator` target will change depending on
 the current allocator flag, so you can unconditionally depend on this
 target to pick up the current build defaults.
 
 This:
 
 ```
   ['use_allocator!='none'', {
     'dependencies': [ '../base/allocator/allocator.gyp:allocator' ]
   }],
   ['win_use_allocator_shim==1', {
     'dependencies': [ '<(allocator_target)' ],
   }],
 ```
 
 Becomes:
 
 ```
   deps = [ "//base/allocator" ]
 ```
 
 As in GYP, the allocator should only be depended on by executables (and
 tests). Libraries should not set the allocator.
 
-### optimize: max 
+### optimize: max
 
 In Gyp:
 
 ```
 'optimize': 'max',
 ```
 
 only affects Windows and will optimize for speed rather than size. To
 get the same behavior in GN, do:
 
 ```
 if (!is_debug && is_win) {
   configs -= [ "//build/config/compiler:optimize" ]
   configs += [ "//build/config/compiler:optimize_max" ]
 }
 ```
 
 The `is_win` check is needed because the `optimize_max` config also
 affects Posix systems. Some components might additionally specify `-O2`
 on Posix further optimize, in which case you can remove the `is_win`
 check.
 
-### Protobufs 
+### Protobufs
 
 ```
 import("//third_party/protobuf/proto_library.gni")
 
 proto_library("myproto") {
   sources = [ "foo.proto" ]
 }
 ```
 
 See the `third_party/protobuf/proto_library.gni` file for full
 documentation and extra flags.
 
-### Java stuff 
+### Java stuff
 
 JNI generator in GYP:
 
 ```
 {
   'target_name': 'foo_headers',
   'type': 'none',
   'sources': [ <java files> ]
   'variables': { 'jni_gen_package': 'foobar' }
   'includes': [ 'build/jni_generator.gypi' ]
@@ skipping 10 matching lines @@
 
 # Later:
 if (is_android) {
   generate_jni("foo_headers") {
     sources = [ <java files> ]
     jni_package = "foobar"
   }
 }
 ```
 
-### Grit 
+### Grit
 
 ```
 import("//tools/grit/grit_rule.gni")
 
 grit("resources") {
   source = "my_resources.grd"
 }
 ```
 
 See `src/build/secondary/tools/grit/grit_rule.gni` for more documentation.
 
-### Mojo 
+### Mojo
 
 ```
 import("//mojo/public/tools/bindings/mojom.gni")
 
 mojom("mojo_bindings") {
   sources = [
     "foo.mojom",
   ]
 }
 ```