New issue
Advanced search Search tips

Issue 919245 link

Starred by 1 user
Status: Assigned
Owner:
dbertoni@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

Update Extensions code to use new base::Value APIs

Project Member Reported by dbertoni@chromium.org, Jan 4 
Move away from base::DictionaryValue, base::ListValue and other deprecated base::Value member functions.

Comment 1 by dbertoni@chromium.org, Jan 9

Description: Show this description

Comment 2 by benhenry@google.com, Jan 11

Labels: Pri-2
Setting defect without priority to default.
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 16 

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

commit 1df6c53603368ebde8b66a830cfa993fb926cdeb
Author: David Bertoni <dbertoni@chromium.org>
Date: Wed Jan 16 03:28:48 2019

[Extensions] All manifest_handler code now uses new base::Value APIs.

Bug: 919245
Change-Id: I69d3a8e68e68d22a79ff76c61ea4c64a06c57c8a
Reviewed-on: https://chromium-review.googlesource.com/c/1396791
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: David Bertoni <dbertoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#623082}
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest.h
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handler_helpers.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handler_helpers.h
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/action_handlers_handler.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/action_handlers_handler_unittest.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/app_isolation_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/background_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/content_scripts_handler.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/file_handler_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/icons_handler.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/icons_handler_unittest.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/kiosk_mode_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/mime_types_handler.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/nacl_modules_handler.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/oauth2_manifest_handler.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/oauth2_manifest_unittest.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/options_page_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/permissions_parser.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/requirements_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/sandboxed_page_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/shared_module_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/web_accessible_resources_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/manifest_handlers/webview_info.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/permissions/api_permission_set.cc
[modify] https://crrev.com/1df6c53603368ebde8b66a830cfa993fb926cdeb/extensions/common/permissions/api_permission_set.h
►

Sign in to add a comment