Move blink::serializedCharacterData array to read-only data segment

Move blink::serializedCharacterData array to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,844 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

BUG=677351
Committed: https://crrev.com/eafbd71d2e7eda209965d66a540d596038edd68c
Cr-Commit-Position: refs/heads/master@{#441083}

[brucedawson, 2016-12-29 21:50:33]

The CQ bit was checked by brucedawson@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-29 21:50:46]

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

[brucedawson, 2016-12-29 21:50:51]

Description was changed from

==========
Move blink::serializedCharacterData arrays to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves about 11,800 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

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

to

==========
Move blink::serializedCharacterData arrays to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves about 11,800 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

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

[brucedawson, 2016-12-29 21:50:51]

brucedawson@chromium.org changed reviewers:
+ jochen@chromium.org

[brucedawson, 2016-12-29 21:52:21]

Description was changed from

==========
Move blink::serializedCharacterData arrays to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves about 11,800 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

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

to

==========
Move blink::serializedCharacterData arrays to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,840 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

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

[brucedawson, 2016-12-29 22:02:32]

Description was changed from

==========
Move blink::serializedCharacterData arrays to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,840 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

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

to

==========
Move blink::serializedCharacterData array to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,840 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

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

[commit-bot: I haz the power, 2016-12-29 23:26:26]

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

[commit-bot: I haz the power, 2016-12-29 23:26:27]

Dry run: This issue passed the CQ dry run.

[brucedawson, 2016-12-29 23:37:10]

The CQ bit was checked by brucedawson@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-29 23:37:22]

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

[brucedawson, 2016-12-29 23:37:56]

Description was changed from

==========
Move blink::serializedCharacterData array to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,840 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

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

to

==========
Move blink::serializedCharacterData array to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,844 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

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

[brucedawson, 2016-12-29 23:38:12]

PTAL

[commit-bot: I haz the power, 2016-12-30 01:08:12]

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

[commit-bot: I haz the power, 2016-12-30 01:08:13]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2017-01-02 07:48:42]

lgtm

[brucedawson, 2017-01-02 23:22:05]

The CQ bit was checked by brucedawson@chromium.org

[commit-bot: I haz the power, 2017-01-02 23:22:16]

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-01-03 01:15:29]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483399325393470,
"parent_rev": "b4c6ec9a0bd0258f0c3b392de0e92940df687e2c", "commit_rev":
"b3e6178942726ebcf6fa595bf3019c9068772877"}

[commit-bot: I haz the power, 2017-01-03 01:16:38]

Description was changed from

==========
Move blink::serializedCharacterData array to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,844 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

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

to

==========
Move blink::serializedCharacterData array to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,844 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

BUG=677351

Review-Url: https://codereview.chromium.org/2608823002
==========

[commit-bot: I haz the power, 2017-01-03 01:16:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-03 01:18:57]

Description was changed from

==========
Move blink::serializedCharacterData array to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,844 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

BUG=677351

Review-Url: https://codereview.chromium.org/2608823002
==========

to

==========
Move blink::serializedCharacterData array to read-only data segment

This array is logically const but was not marked as such. Marking it as
const lets the compiler/linker store it in the read-only data segment,
which is strictly better than being in read/write memory. This change
moves 11,844 bytes from the .data to .rdata segment.

Note that "const" implies static which is why "extern const" is needed.

BUG=677351

Committed: https://crrev.com/eafbd71d2e7eda209965d66a540d596038edd68c
Cr-Commit-Position: refs/heads/master@{#441083}
==========

[commit-bot: I haz the power, 2017-01-03 01:18:58]

Patchset 2 (id:??) landed as
https://crrev.com/eafbd71d2e7eda209965d66a540d596038edd68c
Cr-Commit-Position: refs/heads/master@{#441083}