Attention is currently required from: Mark Mentovai.
Patch set 1:Auto-Submit +1Owners-Override +1
1 comment:
Patchset:
Let me know if you want this split.
To view, visit change 3884228. To unsubscribe, or for help writing mail filters, visit settings.
Attention is currently required from: Avi Drissman, Mark Mentovai.
Patch set 1:Code-Review +1
1 comment:
Patchset:
rs lgtm, spot-checked a couple
To view, visit change 3884228. To unsubscribe, or for help writing mail filters, visit settings.
Attention is currently required from: Mark Mentovai.
Patch set 1:Auto-Submit +1Owners-Override +1
To view, visit change 3888258. To unsubscribe, or for help writing mail filters, visit settings.
Attention is currently required from: Avi Drissman.
Patch set 2:Code-Review +1Commit-Queue +2
19 comments:
Patchset:
I’m going to C-Q this now because I know you’ll want to address all of the concerns I’m raising that you agree need fixing in a follow-up.
File chrome/android/features/start_surface/java/res/values/ids.xml:
44 files including this one have an incorrect blank line in the middle of the boilerplate.
chrome/android/features/start_surface/java/res/values/ids.xml
chrome/android/features/tab_ui/java/res/values/ids.xml
chrome/android/features/tab_ui/java/res/values/styles.xml
chrome/android/feed/core/java/res/values/ids.xml
chrome/android/java/res/anim/accelerate_quart.xml
chrome/android/java/res/anim/activity_close_exit.xml
chrome/android/java/res/anim/activity_open_enter.xml
chrome/android/java/res/anim/decelerate_quart.xml
chrome/android/java/res/drawable-ldrtl-v17/btn_back.xml
chrome/android/java/res/drawable-ldrtl-v17/btn_forward.xml
chrome/android/java/res/drawable-v23/distilled_page_pref_background.xml
chrome/android/java/res/drawable/accessibility_tab_switcher_divider.xml
chrome/android/java/res/drawable/btn_back.xml
chrome/android/java/res/drawable/btn_bg_holo.xml
chrome/android/java/res/drawable/btn_bg_holo_active.xml
chrome/android/java/res/drawable/btn_forward.xml
chrome/android/java/res/drawable/distilled_page_prefs_button_bg.xml
chrome/android/java/res/drawable/store_locally_tooltip_background.xml
chrome/android/java/res/layout/accessibility_tab_switcher.xml
chrome/android/java/res/layout/accessibility_tab_switcher_list_item.xml
chrome/android/java/res/layout/autofill_card_unmask_prompt.xml
chrome/android/java/res/layout/distilled_page_prefs_view.xml
chrome/android/java/res/layout/fre_tosanduma.xml
chrome/android/java/res/layout/lightweight_fre_tos.xml
chrome/android/java/res/layout/password_generation_popup_explanation.xml
chrome/android/java/res/layout/suspended_tab.xml
chrome/android/java/res/values-h600dp/dimens.xml
chrome/android/java/res/values-hdpi/dimens.xml
chrome/android/java/res/values-sw600dp/values.xml
chrome/android/java/res/values-xhdpi/dimens.xml
chrome/android/java/res/values/ids.xml
chrome/android/java/res/values/strings.xml
chrome/android/webapk/libs/common/res_splash/layout/webapp_splash_screen_large.xml
chrome/android/webapk/libs/common/res_splash/layout/webapp_splash_screen_no_icon.xml
chrome/android/webapk/shell_apk/res/values-v24/default_enabled_activity.xml
chrome/android/webapk/shell_apk/res/values/default_enabled_activity.xml
chrome/browser/content_creation/notes/internal/android/java/res/values/styles.xml
chrome/browser/search_engines/android/java/res/values/ids.xml
chrome/browser/tab/java/res/layout/sad_tab.xml
chrome/browser/ui/android/appmenu/internal/java/res/layout/icon_row_menu_item.xml
chrome/browser/ui/android/appmenu/internal/java/res/layout/menu_item.xml
chrome/browser/ui/android/appmenu/internal/java/res/layout/menu_item_start_with_icon.xml
chrome/browser/ui/android/appmenu/internal/java/res/layout/title_button_menu_item.xml
chrome/browser/ui/android/omnibox/java/res/values/ids.xml
chrome/browser/ui/android/toolbar/java/res/values/ids.xml
File chrome/android/java/src/org/chromium/chrome/browser/customtabs/content/CustomTabActivityTabProvider.java:
* Copyright 2019 The Chromium Authors
* Use of this source code is governed by a BSD-style license that can be
* found in the LICENSE file.
Our Java source seems to always use `//` comments for the boilerplate, this is a lone outlier (in chrome).
File chrome/browser/chromeos/extensions/default_keyboard_extension_browser_test.cc:
* Copyright 2013 The Chromium Authors
* Use of this source code is governed by a BSD-style license that can be
* found in the LICENSE file.
In 4 files including this one, a very unusual `/* */` comment style for our C++ code.
chrome/browser/chromeos/extensions/default_keyboard_extension_browser_test.cc
chrome/test/data/nacl/exit_status/pm_exit_status_test.cc
chrome/test/data/nacl/manifest_file/irt_manifest_file_test.cc
chrome/test/data/nacl/sysconf_nprocessors_onln/sysconf_nprocessors_onln_test.cc
File chrome/browser/feed/android/java/res/values-night/colors.xml:
Patch Set #1, Line 4: found in the LICENSE file.-->
In 5 files including this one, the `-->` is butted right up against the `.`, which is contrary to the way we usually do these.
chrome/browser/feed/android/java/res/values-night/colors.xml
chrome/browser/feed/android/java/res/values/colors.xml
chrome/browser/resources/browsing_topics/browsing_topics_internals.html
chrome/browser/share/android/java/res/values-night/colors.xml
chrome/browser/share/android/java/res/values/colors.xml
File chrome/browser/resources/gaia_auth_host/saml_username_autofill_test.unitjs:
File chrome/browser/resources/invalidations/BUILD.gn:
Patch Set #1, Line 3: # found in the LICENSE file
In 2 files including this one, this line does not end with a period as it’s supposed to.
chrome/browser/resources/invalidations/BUILD.gn
chrome/browser/resources/net_internals/BUILD.gn
File chrome/browser/ui/android/appmenu/internal/test/java/res/layout/test_menu_footer.xml:
Patch Set #1, Line 1: <?xml version="1.0" encoding="utf-8"?><!-- Copyright 2019 The Chromium Authors
The boilerplate comment is butted up against the `<?xml>`, unusual.
File chrome/browser/ui/webui/signin/signin_ui_error.cc:
Patch Set #1, Line 1: /// Copyright 2021 The Chromium Authors
In 4 files including this one, too many slashes on this line:
chrome/browser/ui/webui/signin/signin_ui_error.cc
chrome/browser/ui/webui/signin/signin_ui_error.h
chrome/test/base/profile_waiter.cc
chrome/test/base/profile_waiter.h
File chrome/chrome_cleaner/test/resources/resource_ids:
Patch Set #1, Line 1: # Copyright 2014 The Chromium Authors
This file has an incomplete boilerplate.
File chrome/common/chrome_version.h.in:
Patch Set #1, Line 2: // Use of this source is governed by a BSD-style license that can be
This line is missing the word “code” after “source”.
File chrome/common/win/eventlog_messages.mc and b/chrome/common/win/eventlog_messages.mc:
File chrome/test/android/browsertests_apk/AndroidManifest.xml.jinja2:
There’s a blank line in the middle of the boilerplate.
File chrome/test/data/cast/cast_mirroring_performance_browsertest.css:
Patch Set #1, Line 2: * Use of this source code is governed by a BSD-style license that can be
There’s an extra space before “Use”.
File chrome/test/data/extensions/api_test/content_settings/incognitoisolation/test.html:
* Use of this source code is governed by a BSD-style license that can be found
* in the LICENSE file.
In 4 files including this one, this line was rewrapped contrary to our standard. The break should be after the word “be”.
chrome/test/data/extensions/api_test/content_settings/incognitoisolation/test.html
chrome/test/data/extensions/api_test/webrequest/test_simple_cancel.html
chrome/test/data/extensions/api_test/webrequest/test_simple_cancel_navigation.html
chrome/test/data/extensions/api_test/webrequest/test_simple_websocket_cancel.html
File chrome/test/data/extensions/api_test/executescript/bad_encoding/bad_encoding.js:
Tools assuming UTF-8 don’t like this file, so I manually reviewed.
File chrome/test/data/extensions/calculator_app/LICENSE:
Patch Set #1, Line 13: // * Neither the name of Google Inc. nor the names of its
In 2 files including this one:
Google LLC, maybe, or see https://chromium-review.googlesource.com/c/chromium/src/+/3887619/comments/2d58db19_9def9f44.
chrome/test/data/extensions/calculator_app/LICENSE
chrome/test/data/extensions/calculator_app/app/LICENSE
File chrome/test/data/extensions/platform_apps/web_view/dom_storage_isolation/page.html:
Patch Set #1, Line 3: found in the LICENSE file.
This line ends with whitespace.
File chrome/test/data/nacl/exit_status/pm_exit_status_test.html:
Patch Set #1, Line 3: Use of this source code is governed by a BSD-style license that can
In 3 files including this one, this line was rewrapped contrary to our standard. The break should be after the word “be”.
chrome/test/data/nacl/exit_status/pm_exit_status_test.html
chrome/test/data/nacl/manifest_file/irt_manifest_file_test.html
chrome/test/data/nacl/sysconf_nprocessors_onln/sysconf_nprocessors_onln_test.html
To view, visit change 3888258. To unsubscribe, or for help writing mail filters, visit settings.
Attention is currently required from: Mark Mentovai.
Patch set 3:Auto-Submit +1Owners-Override +1
To view, visit change 3888258. To unsubscribe, or for help writing mail filters, visit settings.
Attention is currently required from: Avi Drissman.
Patch set 3:Code-Review +1Commit-Queue +2
1 comment:
Patchset:
Noted the removal in patch set 3 of:
chrome/browser/plugins/plugin_finder.h
chrome/browser/plugins/plugin_finder.cc
chrome/browser/plugins/plugin_finder_unittest.cc
chrome/browser/plugins/plugin_infobar_delegates.h
chrome/browser/plugins/plugin_infobar_delegates.cc
chrome/browser/plugins/plugin_installer.h
chrome/browser/plugins/plugin_installer.cc
chrome/browser/plugins/plugin_installer_observer.h
chrome/browser/plugins/plugin_installer_observer.cc
chrome/browser/plugins/plugin_installer_unittest.cc
Attention is currently required from: KAMPANAT THUMWONG.
1 comment:
Patchset:
kongp...@gmail.com+Xauth0918600717
To view, visit change 3888258. To unsubscribe, or for help writing mail filters, visit settings.
Attention is currently required from: Avi Drissman, Daniel Cheng, KAMPANAT THUMWONG, Sadrul Chowdhury.
1 comment:
File chrome/browser/chromeos/extensions/default_keyboard_extension_browser_test.cc:
* Copyright 2013 The Chromium Authors
* Use of this source code is governed by a BSD-style license that can be
* found in the LICENSE file.
In 4 files including this one, a very unusual `/* */` comment style for our C++ code. […]
To view, visit change 3888258. To unsubscribe, or for help writing mail filters, visit settings.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |