|
|
Created:
3 years, 7 months ago by yoichio Modified:
3 years, 7 months ago Reviewers:
yosin_UTC9 CC:
blink-reviews, chromium-reviews Target Ref:
refs/heads/master Project:
chromium Visibility:
Public. |
DescriptionMake for loop using C++11 foreach in LayoutSelection::SetSelection()
BUG=708453
Review-Url: https://codereview.chromium.org/2876363002
Cr-Commit-Position: refs/heads/master@{#471977}
Committed: https://chromium.googlesource.com/chromium/src/+/56fd91872a4109611d4c1142243b349e430b61a2
Patch Set 1 #
Total comments: 2
Patch Set 2 : update #Messages
Total messages: 20 (12 generated)
The CQ bit was checked by yoichio@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
yoichio@chromium.org changed reviewers: + yosin@chromium.org
https://codereview.chromium.org/2876363002/diff/1/third_party/WebKit/Source/c... File third_party/WebKit/Source/core/editing/LayoutSelection.cpp (right): https://codereview.chromium.org/2876363002/diff/1/third_party/WebKit/Source/c... third_party/WebKit/Source/core/editing/LayoutSelection.cpp:223: for (const auto& it : old_selected_map.object_map) nit: s/it/key_value/ or s/it/layout_object_and_selection_state/ Using |it| is confusing, since |it| is used for iterator. range-for is short-form of for (auto it = map.begin(); it != map.end(); ++it) { const auto& key_value = *it; ... }
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
https://codereview.chromium.org/2876363002/diff/1/third_party/WebKit/Source/c... File third_party/WebKit/Source/core/editing/LayoutSelection.cpp (right): https://codereview.chromium.org/2876363002/diff/1/third_party/WebKit/Source/c... third_party/WebKit/Source/core/editing/LayoutSelection.cpp:223: for (const auto& it : old_selected_map.object_map) On 2017/05/15 07:05:09, yosin_UTC9 wrote: > nit: s/it/key_value/ or s/it/layout_object_and_selection_state/ > > Using |it| is confusing, since |it| is used for iterator. > > range-for is short-form of > > for (auto it = map.begin(); it != map.end(); ++it) { > const auto& key_value = *it; > ... > } Done.
The CQ bit was checked by yoichio@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
lgtm
Nice readability improvement! :)
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by yoichio@chromium.org
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch. Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494898402369780, "parent_rev": "9847aeaf079339c729859157552eabde17ae3b17", "commit_rev": "56fd91872a4109611d4c1142243b349e430b61a2"}
Message was sent while issue was closed.
Description was changed from ========== Make for loop using C++11 foreach in LayoutSelection::SetSelection() BUG=708453 ========== to ========== Make for loop using C++11 foreach in LayoutSelection::SetSelection() BUG=708453 Review-Url: https://codereview.chromium.org/2876363002 Cr-Commit-Position: refs/heads/master@{#471977} Committed: https://chromium.googlesource.com/chromium/src/+/56fd91872a4109611d4c1142243b... ==========
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as https://chromium.googlesource.com/chromium/src/+/56fd91872a4109611d4c1142243b... |