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

Issue 14307012: webkit: Remove use of ALLOW_THIS_IN_INITIALIZER_LIST. (Closed)

Created:
7 years, 8 months ago by scherkus (not reviewing)
Modified:
7 years, 7 months ago
Reviewers:
jamesr
CC:
chromium-reviews, michaeln, vsevik, tzik+watch_chromium.org, yurys, darin-cc_chromium.org, sail+watch_chromium.org, oshima+watch_chromium.org, kinuko+watch, stevenjb+watch_chromium.org, pfeldman
Visibility:
Public.

Description

webkit: Remove use of ALLOW_THIS_IN_INITIALIZER_LIST. It's no longer providing value as the MSVC warning is disabled during compilation. Refer to bug for details. BUG=234765 R=jamesr@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=197204

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+78 lines, -81 lines) Patch
M webkit/appcache/appcache_quota_client_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/appcache/appcache_response.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M webkit/appcache/appcache_service_unittest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M webkit/appcache/appcache_storage_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/appcache/appcache_update_job.cc View 1 chunk +1 line, -2 lines 0 comments Download
M webkit/appcache/appcache_url_request_job.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/appcache/mock_appcache_storage.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/appcache/view_appcache_internals_job.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M webkit/appcache/web_application_cache_host_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/blob/blob_url_request_job.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/blob/local_file_stream_reader.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/blob/view_blob_internals_job.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/chromeos/fileapi/remote_file_stream_writer.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/database/database_quota_client_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/dom_storage/dom_storage_cached_area.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/file_system_dir_url_request_job.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/file_system_dir_url_request_job_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/file_system_file_stream_reader.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/file_system_mount_point_provider_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/file_system_quota_client_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/file_system_url_request_job.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/file_system_url_request_job_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/file_system_usage_cache.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/file_writer_delegate.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/local_file_stream_writer.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/local_file_system_operation.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/local_file_system_quota_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/obfuscated_file_util_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/sandbox_file_stream_writer.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/sandbox_mount_point_provider.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/sandbox_quota_observer.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/syncable/syncable_file_operation_runner_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/syncable/syncable_file_system_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/fileapi/upload_file_system_file_element_reader.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/glue/websocketstreamhandle_impl.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M webkit/glue/weburlloader_impl.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/npapi/webplugin_delegate_impl_win.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/npapi/webplugin_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/ppapi/content_decryptor_delegate.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/ppapi/message_channel.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/ppapi/ppapi_plugin_instance.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/ppapi/ppb_graphics_2d_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/ppapi/ppb_graphics_3d_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/ppapi/ppb_scrollbar_impl.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/ppapi/quota_file_io.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M webkit/plugins/ppapi/quota_file_io_unittest.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M webkit/quota/mock_quota_manager.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/quota/mock_quota_manager_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/quota/mock_storage_client.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/quota/quota_manager.cc View 6 chunks +6 lines, -6 lines 0 comments Download
M webkit/quota/quota_manager_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/quota/quota_temporary_storage_evictor.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/quota/quota_temporary_storage_evictor_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/quota/usage_tracker.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M webkit/tools/test_shell/simple_appcache_system.cc View 1 1 chunk +2 lines, -4 lines 0 comments Download
M webkit/tools/test_shell/test_shell_devtools_agent.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/tools/test_shell/test_shell_devtools_client.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/tools/test_shell/test_shell_request_context.cc View 1 2 chunks +2 lines, -2 lines 0 comments Download
M webkit/tools/test_shell/webwidget_host_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/tools/test_shell/webwidget_host_mac.mm View 1 chunk +1 line, -1 line 0 comments Download
M webkit/tools/test_shell/webwidget_host_win.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
scherkus (not reviewing)
7 years, 8 months ago (2013-04-26 20:56:50 UTC) #1
jamesr
lgtm
7 years, 8 months ago (2013-04-26 20:58:10 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/scherkus@chromium.org/14307012/2001
7 years, 8 months ago (2013-04-26 21:06:53 UTC) #3
commit-bot: I haz the power
Failed to apply patch for webkit/plugins/ppapi/ppb_graphics_2d_impl.cc: While running patch -p1 --forward --force --no-backup-if-mismatch; can't find ...
7 years, 8 months ago (2013-04-27 01:09:38 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/scherkus@chromium.org/14307012/2001
7 years, 8 months ago (2013-04-27 01:37:54 UTC) #5
commit-bot: I haz the power
Failed to apply patch for webkit/plugins/ppapi/ppb_graphics_2d_impl.cc: While running patch -p1 --forward --force --no-backup-if-mismatch; can't find ...
7 years, 8 months ago (2013-04-27 01:38:05 UTC) #6
scherkus (not reviewing)
7 years, 7 months ago (2013-04-30 01:05:52 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 manually as r197204 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698