| commit | f37e7fd6eb873016a023acc58a121161128aeabd | [log] [tgz] |
|---|---|---|
| author | rlanday <rlanday@chromium.org> | Fri Jul 07 20:52:34 2017 |
| committer | Commit Bot <commit-bot@chromium.org> | Fri Jul 07 20:52:34 2017 |
| tree | 64bc9c70256cd30e489b2610dffde07ee50e08bc | |
| parent | 54c314a569da0a5f5c61436a36a29969fbfe2871 [diff] |
Clean up DocumentMarkerController::MarkersFor() I'm cleaning up two things here: - Adding a check to PossiblyHasMarkers() (probably doesn't save us very much time, but we can at least avoid a hashmap lookup in the "no markers" case) - Fixing the loop to just loop over the passed-in set of MarkerTypes vs. looping over all types and skipping over some of them BUG=707867 Review-Url: https://codereview.chromium.org/2956453002 Cr-Commit-Position: refs/heads/master@{#485038}