Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(611)

Issue 13165005: Move FileEnumerator to its own file, do some refactoring. (Closed)

Created:
7 years, 8 months ago by brettw
Modified:
7 years, 7 months ago
CC:
chromium-reviews, tzik+watch_chromium.org, jam, nkostylev+watch_chromium.org, achuith+watch_chromium.org, amit, ahutter, browser-components-watch_chromium.org, jln+watch_chromium.org, chrome-speed-team+watch_google.com, rginda+watch_chromium.org, Raman Kakilate, yfriedman+watch_chromium.org, markusheintz_, stevenjb+watch_chromium.org, kinuko+watch, frankf+watch_chromium.org, dcaiafa+watch_chromium.org, cbentzel+watch_chromium.org, gavinp+disk_chromium.org, benquan, grt+watch_chromium.org, kkania, dbeam+watch-autofill_chromium.org, apatrick_chromium, joi+watch-content_chromium.org, darin-cc_chromium.org, bulach+watch_chromium.org, timurrrr+watch_chromium.org, dhollowa+watch_chromium.org, chromium-apps-reviews_chromium.org, erikwright+watch_chromium.org, Dane Wallinga, derat+watch_chromium.org, wez+watch_chromium.org, sanjeevr, simonmorris+watch_chromium.org, feature-media-reviews_chromium.org, rmsousa+watch_chromium.org, oshima+watch_chromium.org, Albert Bodenhamer, sergeyu+watch_chromium.org, ilevy+watch_chromium.org, jamiewalch+watch_chromium.org, hclam+watch_chromium.org, Ilya Sherman, tfarina, klundberg+watch_chromium.org, sail+watch_chromium.org, garykac+watch_chromium.org, Aaron Boodman, lambroslambrou+watch_chromium.org, robertshield, davemoore+watch_chromium.org, dyu1, estade+watch_chromium.org, alexeypa+watch_chromium.org, bruening+watch_chromium.org, glider+watch_chromium.org, jshin+watch_chromium.org
Visibility:
Public.

Description

Move FileEnumerator to its own file, do some refactoring. It creates a class FileInfo to contain the details rather than using a platform-specific typedef. This allows the accessors GetName, GetSize, etc. to be moved directly to this class (previously they were static helpers on the FileEnumerator class) which makes a bunch of code much cleaner. It also gives reasonable getting and initialization which the previous version lacked. BUG=175002 R=rvargas@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=198820

Patch Set 1 #

Patch Set 2 : merge #

Patch Set 3 : fixes #

Patch Set 4 : #

Patch Set 5 : #

Patch Set 6 : #

Patch Set 7 : mergey-wergey #

Patch Set 8 : merge, fixes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+973 lines, -912 lines) Patch
M base/base.gypi View 1 2 3 4 5 6 7 2 chunks +5 lines, -0 lines 0 comments Download
M base/file_util.h View 1 2 3 3 chunks +3 lines, -111 lines 0 comments Download
M base/file_util.cc View 1 2 3 4 chunks +5 lines, -24 lines 0 comments Download
M base/file_util_posix.cc View 1 2 3 10 chunks +13 lines, -165 lines 0 comments Download
M base/file_util_unittest.cc View 1 2 3 14 chunks +20 lines, -22 lines 0 comments Download
M base/file_util_win.cc View 1 2 3 1 chunk +0 lines, -133 lines 0 comments Download
A base/files/file_enumerator.h View 1 2 3 4 1 chunk +156 lines, -0 lines 0 comments Download
A base/files/file_enumerator.cc View 1 2 3 1 chunk +21 lines, -0 lines 0 comments Download
A base/files/file_enumerator_posix.cc View 1 1 chunk +160 lines, -0 lines 0 comments Download
A base/files/file_enumerator_win.cc View 1 2 3 1 chunk +149 lines, -0 lines 0 comments Download
M base/test/test_file_util_posix.cc View 1 2 3 1 chunk +1 line, -2 lines 0 comments Download
M chrome/browser/browsing_data/browsing_data_local_storage_helper_browsertest.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/camera_detector.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/contacts/contact_database_unittest.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/drive/file_cache.cc View 1 2 3 4 5 6 7 3 chunks +5 lines, -8 lines 0 comments Download
M chrome/browser/chromeos/drive/file_cache_metadata.cc View 1 2 3 4 5 6 7 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/drive/file_cache_unittest.cc View 1 2 3 4 5 6 7 4 chunks +8 lines, -9 lines 0 comments Download
M chrome/browser/chromeos/extensions/wallpaper_private_api.cc View 3 chunks +5 lines, -4 lines 0 comments Download
M chrome/browser/chromeos/login/wallpaper_manager.cc View 1 2 3 4 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/policy/app_pack_updater.cc View 1 2 3 4 3 chunks +6 lines, -8 lines 0 comments Download
M chrome/browser/component_updater/pepper_flash_component_installer.cc View 1 2 3 4 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/component_updater/pnacl/pnacl_component_installer.cc View 1 2 3 4 5 6 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/component_updater/swiftshader_component_installer.cc View 1 2 3 4 5 6 7 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/component_updater/widevine_cdm_component_installer.cc View 1 2 3 4 5 6 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/extensions/api/developer_private/developer_private_api.cc View 1 2 3 4 5 6 7 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/extensions/api/serial/serial_port_enumerator_posix.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M chrome/browser/extensions/api/storage/settings_backend.cc View 1 2 3 4 2 chunks +4 lines, -7 lines 0 comments Download
M chrome/browser/extensions/extension_service_unittest.cc View 1 2 3 4 5 6 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/extensions/external_pref_loader.cc View 1 2 3 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/extensions/sandboxed_unpacker_unittest.cc View 1 2 3 4 5 6 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/browser/file_select_helper.cc View 1 2 chunks +2 lines, -1 line 0 comments Download
M chrome/browser/history/expire_history_backend.cc View 1 2 3 4 5 6 7 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/history/text_database_manager.cc View 1 2 3 4 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/importer/firefox2_importer.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/browser/importer/firefox3_importer.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/importer/ie_importer.cc View 1 2 3 4 5 6 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/policy/config_dir_policy_loader.cc View 1 2 3 3 chunks +5 lines, -4 lines 0 comments Download
M chrome/browser/policy/policy_browsertest.cc View 1 2 3 4 5 6 3 chunks +5 lines, -5 lines 0 comments Download
M chrome/browser/printing/printing_layout_browsertest.cc View 1 2 3 4 5 6 7 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/profiles/profile_manager.cc View 1 2 3 4 5 6 7 2 chunks +5 lines, -8 lines 0 comments Download
M chrome/browser/profiles/profile_shortcut_manager_win.cc View 1 2 3 4 5 6 7 3 chunks +5 lines, -4 lines 0 comments Download
M chrome/browser/shell_integration_win.cc View 1 2 3 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/ui/pdf/pdf_browsertest.cc View 1 2 3 4 5 6 7 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/ui/webui/chromeos/drive_internals_ui.cc View 1 2 3 4 5 6 7 3 chunks +11 lines, -14 lines 0 comments Download
M chrome/browser/ui/webui/feedback_ui.cc View 1 2 3 4 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/common/extensions/api/i18n/default_locale_handler.cc View 2 chunks +2 lines, -3 lines 0 comments Download
M chrome/common/extensions/extension_file_util.cc View 1 2 3 5 chunks +10 lines, -10 lines 0 comments Download
M chrome/common/extensions/extension_l10n_util.cc View 1 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/common/extensions/unpacker.cc View 1 2 3 4 5 6 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/installer/setup/setup_util.cc View 1 2 3 4 3 chunks +6 lines, -6 lines 0 comments Download
M chrome/installer/setup/uninstall.cc View 1 2 3 4 5 6 4 chunks +7 lines, -9 lines 0 comments Download
M chrome/installer/test/alternate_version_generator.cc View 1 2 3 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/installer/util/delete_after_reboot_helper.cc View 3 chunks +5 lines, -4 lines 0 comments Download
M chrome/installer/util/duplicate_tree_detector.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/installer/util/installer_state.cc View 1 2 3 4 5 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/installer/util/installer_state_unittest.cc View 1 2 3 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/installer/util/shell_util.cc View 1 2 3 4 5 6 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/installer/util/shell_util_unittest.cc View 1 2 3 4 5 6 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/test/automation/proxy_launcher.cc View 1 2 3 4 5 6 7 3 chunks +6 lines, -8 lines 0 comments Download
M chrome/test/chromedriver/chrome/zip.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/test/chromedriver/util.cc View 1 2 3 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/test/gpu/gpu_pixel_browsertest.cc View 1 2 3 2 chunks +5 lines, -4 lines 0 comments Download
M chrome/test/mini_installer_test/installer_path_provider.cc View 7 chunks +13 lines, -24 lines 0 comments Download
M chrome/test/perf/page_cycler_test.cc View 1 3 chunks +5 lines, -4 lines 0 comments Download
M chrome/test/perf/startup_test.cc View 1 2 3 4 5 6 1 chunk +1 line, -0 lines 0 comments Download
M chrome/test/reliability/automated_ui_tests.cc View 1 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/test/reliability/page_load_test.cc View 1 2 3 4 chunks +10 lines, -9 lines 0 comments Download
M chrome/test/ui/ui_test.cc View 1 2 3 4 5 6 3 chunks +4 lines, -5 lines 0 comments Download
M chrome/test/webdriver/webdriver_util.cc View 1 2 3 4 5 6 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/tools/profiles/generate_profile.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M chrome_frame/test/reliability/page_load_test.cc View 1 2 3 3 chunks +7 lines, -6 lines 0 comments Download
M cloud_print/virtual_driver/win/port_monitor/port_monitor.cc View 1 2 3 2 chunks +3 lines, -5 lines 0 comments Download
M cloud_print/virtual_driver/win/virtual_driver.gypi View 1 2 3 4 1 chunk +1 line, -0 lines 0 comments Download
M components/autofill/browser/data_driven_test.cc View 1 2 3 2 chunks +5 lines, -4 lines 0 comments Download
M content/browser/gpu/gpu_pixel_browsertest.cc View 1 2 3 2 chunks +5 lines, -4 lines 0 comments Download
M content/browser/indexed_db/indexed_db_context_impl.cc View 1 2 3 4 5 6 2 chunks +3 lines, -2 lines 0 comments Download
M content/browser/renderer_host/pepper/pepper_flash_file_message_filter.cc View 2 chunks +8 lines, -8 lines 0 comments Download
M content/browser/storage_partition_impl_map.cc View 1 2 3 4 chunks +8 lines, -7 lines 0 comments Download
M content/browser/zygote_host/zygote_host_impl_linux.cc View 1 2 3 4 2 chunks +2 lines, -2 lines 0 comments Download
M content/gpu/gpu_info_collector_win.cc View 1 2 3 4 5 6 2 chunks +3 lines, -2 lines 0 comments Download
M content/test/image_decoder_test.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M gpu/tools/compositor_model_bench/compositor_model_bench.cc View 1 2 3 3 chunks +2 lines, -2 lines 0 comments Download
M media/video/capture/linux/video_capture_device_linux.cc View 1 2 3 3 chunks +6 lines, -6 lines 0 comments Download
M net/base/directory_lister.h View 2 chunks +2 lines, -2 lines 0 comments Download
M net/base/directory_lister.cc View 1 2 3 3 chunks +16 lines, -28 lines 0 comments Download
M net/base/directory_lister_unittest.cc View 1 2 3 4 5 6 7 3 chunks +9 lines, -9 lines 0 comments Download
M net/disk_cache/block_files_unittest.cc View 2 chunks +2 lines, -1 line 0 comments Download
M net/disk_cache/cache_util_posix.cc View 3 chunks +6 lines, -6 lines 0 comments Download
M net/disk_cache/simple/simple_index.cc View 1 2 3 4 5 6 4 chunks +9 lines, -10 lines 0 comments Download
M net/tools/dump_cache/dump_files.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M net/tools/quic/quic_in_memory_cache.cc View 1 2 3 4 2 chunks +4 lines, -4 lines 0 comments Download
M net/url_request/url_request_file_dir_job.cc View 1 2 3 1 chunk +8 lines, -18 lines 0 comments Download
M remoting/host/policy_hack/policy_watcher_linux.cc View 1 2 3 3 chunks +6 lines, -5 lines 0 comments Download
M third_party/leveldatabase/env_chromium.cc View 1 2 3 4 5 6 2 chunks +3 lines, -2 lines 0 comments Download
M third_party/zlib/google/zip.cc View 1 2 3 4 5 6 2 chunks +3 lines, -3 lines 0 comments Download
M third_party/zlib/google/zip_unittest.cc View 1 2 3 4 5 6 2 chunks +3 lines, -3 lines 0 comments Download
M tools/android/md5sum/md5sum.cc View 1 2 3 2 chunks +3 lines, -2 lines 0 comments Download
M tools/heapcheck/suppressions.txt View 1 2 3 4 5 6 1 chunk +2 lines, -2 lines 0 comments Download
M tools/valgrind/tsan/suppressions.txt View 1 1 chunk +2 lines, -2 lines 0 comments Download
M webkit/database/database_tracker.cc View 3 chunks +5 lines, -4 lines 0 comments Download
M webkit/dom_storage/dom_storage_context.cc View 1 2 3 3 chunks +6 lines, -8 lines 0 comments Download
M webkit/fileapi/file_system_database_test_helper.cc View 1 2 3 3 chunks +5 lines, -6 lines 0 comments Download
M webkit/fileapi/file_system_directory_database.cc View 1 2 3 4 3 chunks +4 lines, -7 lines 0 comments Download
M webkit/fileapi/file_system_origin_database.cc View 1 2 3 4 2 chunks +4 lines, -3 lines 0 comments Download
M webkit/fileapi/isolated_file_util_unittest.cc View 1 2 3 4 5 6 7 3 chunks +7 lines, -9 lines 0 comments Download
M webkit/fileapi/local_file_util.cc View 1 2 3 5 chunks +8 lines, -8 lines 0 comments Download
M webkit/fileapi/native_file_util.cc View 1 2 3 3 chunks +8 lines, -8 lines 0 comments Download
M webkit/plugins/npapi/plugin_list_mac.mm View 2 chunks +4 lines, -3 lines 0 comments Download
M webkit/plugins/npapi/plugin_list_posix.cc View 1 2 3 2 chunks +4 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
brettw
Most of the change is just mechanical search-and-replace file_util:: -> base:: and fixing up the ...
7 years, 7 months ago (2013-05-03 18:10:13 UTC) #1
rvargas (doing something else)
Thanks for doing this. LGTM
7 years, 7 months ago (2013-05-06 19:44:04 UTC) #2
brettw
7 years, 7 months ago (2013-05-07 22:51:47 UTC) #3
Message was sent while issue was closed.
Committed patchset #8 manually as r198820.

Powered by Google App Engine
This is Rietveld 408576698