Make GRIT not to generate const struct members

Make GRIT not to generate const struct members

chrome.dll:
       .text:      16 bytes change
      .rdata:   25376 bytes change
       .data:  -25536 bytes change
      .reloc:     128 bytes change
Total change:     -16 bytes

BUG=677351
Review-Url: https://codereview.chromium.org/2685073002
Cr-Commit-Position: refs/heads/master@{#453145}
Committed: https://chromium.googlesource.com/chromium/src/+/b7c41be9c2235fa51c1e6e0abd3963d277cc1df2

[sabbakumov, 2017-02-09 02:49:18]

Description was changed from

==========
Make GRIT not to generate const struct members

chrome.dll:
       .text:      16 bytes change
      .rdata:   25376 bytes change
       .data:  -25536 bytes change
      .reloc:     128 bytes change
Total change:     -16 bytes

BUG=677351
==========

to

==========
Make GRIT not to generate const struct members

chrome.dll:
       .text:      16 bytes change
      .rdata:   25376 bytes change
       .data:  -25536 bytes change
      .reloc:     128 bytes change
Total change:     -16 bytes

BUG=677351
==========

[sabbakumov, 2017-02-09 02:49:18]

sabbakumov@yandex-team.ru changed reviewers:
+ thestig@chromium.org

[Lei Zhang, 2017-02-23 19:33:28]

lgtm

[Lei Zhang, 2017-02-23 19:34:28]

Please note the status message embedded in user names. I was away for a while,
so you could have gotten a review in a more timely manner from another person in
the grit OWNERS file.

[sabbakumov, 2017-02-27 03:17:08]

The CQ bit was checked by sabbakumov@yandex-team.ru to run a CQ dry run

[commit-bot: I haz the power, 2017-02-27 03:17:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-27 04:10:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-27 04:10:57]

Dry run: This issue passed the CQ dry run.

[sabbakumov, 2017-02-27 04:27:01]

The CQ bit was checked by sabbakumov@yandex-team.ru

[commit-bot: I haz the power, 2017-02-27 04:27:26]

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

[commit-bot: I haz the power, 2017-02-27 04:31:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488169621540740, "parent_rev":
"a4712e64ac30b14b093c62edd1c238b06d34de4b", "commit_rev":
"b7c41be9c2235fa51c1e6e0abd3963d277cc1df2"}

[commit-bot: I haz the power, 2017-02-27 04:32:37]

Description was changed from

==========
Make GRIT not to generate const struct members

chrome.dll:
       .text:      16 bytes change
      .rdata:   25376 bytes change
       .data:  -25536 bytes change
      .reloc:     128 bytes change
Total change:     -16 bytes

BUG=677351
==========

to

==========
Make GRIT not to generate const struct members

chrome.dll:
       .text:      16 bytes change
      .rdata:   25376 bytes change
       .data:  -25536 bytes change
      .reloc:     128 bytes change
Total change:     -16 bytes

BUG=677351

Review-Url: https://codereview.chromium.org/2685073002
Cr-Commit-Position: refs/heads/master@{#453145}
Committed:
https://chromium.googlesource.com/chromium/src/+/b7c41be9c2235fa51c1e6e0abd39...
==========

[commit-bot: I haz the power, 2017-02-27 04:32:38]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b7c41be9c2235fa51c1e6e0abd39...

[brucedawson, 2017-02-27 18:41:12]

Hey, that's great. That moves a *lot* of globals to the read-only data segment.
I think that now the vast majority of the globals that should be in the
read-only segment are, although I'm still going to run another report just to
see, with tomorrow's canary.