Add braces to single-line if statements in Page Info Code |
||||
Issue descriptionno goto fail plz
,
Mar 14 2017
Assigning to you for now lgarron.
,
Jun 9 2017
,
Jun 22 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1e5352e7505feac546757f8bcdd7217b09a136a0 commit 1e5352e7505feac546757f8bcdd7217b09a136a0 Author: sperigo <sperigo@chromium.org> Date: Thu Jun 22 02:28:48 2017 Page Info: Add brackets in to one line if and for statements BUG= 699840 Review-Url: https://codereview.chromium.org/2946323003 Cr-Commit-Position: refs/heads/master@{#481409} [modify] https://crrev.com/1e5352e7505feac546757f8bcdd7217b09a136a0/chrome/browser/ui/views/page_info/chosen_object_row.cc [modify] https://crrev.com/1e5352e7505feac546757f8bcdd7217b09a136a0/chrome/browser/ui/views/page_info/page_info_bubble_view.cc [modify] https://crrev.com/1e5352e7505feac546757f8bcdd7217b09a136a0/chrome/browser/ui/views/page_info/page_info_bubble_view_unittest.cc [modify] https://crrev.com/1e5352e7505feac546757f8bcdd7217b09a136a0/chrome/browser/ui/views/page_info/permission_selector_row.cc
,
Aug 2 2017
Is this fixed now, or is there more work to do?
,
Aug 2 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by lgar...@chromium.org
, Mar 9 2017