diff options
| author | hiroshige <hiroshige@chromium.org> | 2016-02-03 02:22:39 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-02-03 10:23:38 +0000 |
| commit | 4373b80ebf3b114c9b99deaa7a7b02e362eacc00 (patch) | |
| tree | 3b6a7783abea10efe20a2a476fca48636425c939 /courgette | |
| parent | df3e8ee64956ecfb320dfbd09d6d0092b54687fa (diff) | |
| download | chromium_src-4373b80ebf3b114c9b99deaa7a7b02e362eacc00.zip chromium_src-4373b80ebf3b114c9b99deaa7a7b02e362eacc00.tar.gz chromium_src-4373b80ebf3b114c9b99deaa7a7b02e362eacc00.tar.bz2 | |
Enforce MediaValuesCached instead of MediaValues in some class members
The followings should point to MediaValuesCached:
- StartTagScanner::m_mediaValues
- CachedDocumentParameters::mediaValues
Because:
- isCached() should be true according to ASSERTs.
- isSafeToSendToAnotherThread() should be true because
CachedDocumentParameters::mediaValues is passed across threads.
This CL changes their types from MediaValues to MediaValuesCached to enforce
this and clarify MediaValuesDynamic is not passed across thread,
and remove ASSERTs, isCached(), and isSafeToSendToAnotherThread() that
now become unnecessary.
BUG=581565, 476928
Review URL: https://codereview.chromium.org/1578363010
Cr-Commit-Position: refs/heads/master@{#373209}
Diffstat (limited to 'courgette')
0 files changed, 0 insertions, 0 deletions