New issue
Advanced search Search tips

Issue 709701 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: May 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug
Team-Security-UX

Blocking:
issue 663975



Sign in to add a comment

Remove page info strings only used in dead code.

Project Member Reported by lgar...@chromium.org, Apr 8 2017

Issue description

e.g. IDS_PAGE_INFO_TITLE_SITE_PERMISSIONS
 
Blocking: 663975
IDS_PAGE_INFO_PERMISSION_ALLOW
IDS_PAGE_INFO_PERMISSION_BLOCK
IDS_PAGE_INFO_PERMISSION_ASK
Project Member

Comment 2 by bugdroid1@chromium.org, May 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fc1d5e5df6ef53ecc87135178352b22665296c3a

commit fc1d5e5df6ef53ecc87135178352b22665296c3a
Author: lgarron <lgarron@chromium.org>
Date: Wed May 03 23:15:21 2017

Move all Page Info strings into page_info_strings.grdp

This CL does the following:
- Move strings into page_info_strings.grdp
- Organize the strings into sections with comment headings, based on the order and role they have in the UI.
- Remove old "chrome/grit/generated_resources.h" includes in C++ files where possible.
- Replace IDS_PAGE_INFO_TITLE_SITE_DATA with IDS_PAGE_INFO_COOKIES ( crbug.com/716296 )
- Remove dead strings and their associated code ( crbug.com/709701 )
- Rename remaining IDS_PAGEINFO_* strings to IDS_PAGE_INFO_* ( crbug.com/716305 )
- Unfork IDS_PAGE_INFO_NON_SECURE_TRANSPORT into IDS_PAGEINFO_NOT_SECURE_SUMMARY

BUG= 663975 ,  709701 ,  716296 ,  716296 ,  716305 
TBR=meacer@chromium.org, asanka@chromium.org, kkhorimoto@chromium.org

Review-Url: https://codereview.chromium.org/2849713003
Cr-Commit-Position: refs/heads/master@{#469190}

[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/app/chromium_strings.grd
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/app/generated_resources.grd
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/app/google_chrome_strings.grd
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/android/vr_shell/textures/insecure_content_permanent_texture.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/android/vr_shell/textures/insecure_content_transient_texture.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/android/page_info/connection_info_popup_android.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.mm
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/login/login_handler.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/login/login_handler_unittest.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/page_info/page_info.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/page_info/page_info_infobar_delegate.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/page_info/page_info_ui.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/page_info/page_info_ui.h
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/page_info/permission_menu_model.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/views/certificate_selector.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/chrome/browser/ui/views/page_info/page_info_bubble_view.cc
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/components/components_chromium_strings.grd
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/components/components_google_chrome_strings.grd
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/components/components_strings.grd
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/components/page_info_strings.grdp
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/ios/chrome/browser/ui/dialogs/nsurl_protection_space_util.mm
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/ios/chrome/browser/ui/dialogs/nsurl_protection_space_util_unittest.mm
[modify] https://crrev.com/fc1d5e5df6ef53ecc87135178352b22665296c3a/ios/chrome/browser/ui/omnibox/page_info_model.cc

Status: Fixed (was: Available)

Sign in to add a comment