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

Issue 17476002: delete webkit/support/simple_xxx's (Closed)

Created:
7 years, 6 months ago by scottmg
Modified:
7 years, 6 months ago
Reviewers:
jamesr
CC:
chromium-reviews, darin-cc_chromium.org
Visibility:
Public.

Description

delete webkit/support/simple_xxx's webkit/support/simple_resource_loader_bridge* primarily after mocks fixed, which allows the rest of them which depended on the resource loading to also be deleted. R=jamesr@chromium.org BUG=237267 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=208265

Patch Set 1 #

Total comments: 2

Patch Set 2 : and the rest! #

Patch Set 3 : rebase #

Patch Set 4 : clang warning #

Patch Set 5 : android #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -3962 lines) Patch
M webkit/support/platform_support_android.cc View 1 2 3 4 1 chunk +0 lines, -1 line 0 comments Download
D webkit/support/simple_appcache_system.h View 1 chunk +0 lines, -136 lines 0 comments Download
D webkit/support/simple_appcache_system.cc View 1 chunk +0 lines, -465 lines 0 comments Download
D webkit/support/simple_database_system.h View 1 1 chunk +0 lines, -105 lines 0 comments Download
D webkit/support/simple_database_system.cc View 1 1 chunk +0 lines, -319 lines 0 comments Download
D webkit/support/simple_dom_storage_system.h View 1 1 chunk +0 lines, -79 lines 0 comments Download
D webkit/support/simple_dom_storage_system.cc View 1 1 chunk +0 lines, -286 lines 0 comments Download
D webkit/support/simple_file_system.h View 1 chunk +0 lines, -157 lines 0 comments Download
D webkit/support/simple_file_system.cc View 1 chunk +0 lines, -408 lines 0 comments Download
D webkit/support/simple_file_writer.h View 1 chunk +0 lines, -53 lines 0 comments Download
D webkit/support/simple_file_writer.cc View 1 chunk +0 lines, -201 lines 0 comments Download
D webkit/support/simple_resource_loader_bridge.h View 1 chunk +0 lines, -61 lines 0 comments Download
D webkit/support/simple_resource_loader_bridge.cc View 1 chunk +0 lines, -1169 lines 0 comments Download
D webkit/support/simple_socket_stream_bridge.h View 1 1 chunk +0 lines, -35 lines 0 comments Download
D webkit/support/simple_socket_stream_bridge.cc View 1 1 chunk +0 lines, -259 lines 0 comments Download
D webkit/support/simple_webcookiejar_impl.h View 1 chunk +0 lines, -22 lines 0 comments Download
D webkit/support/simple_webcookiejar_impl.cc View 1 chunk +0 lines, -25 lines 0 comments Download
M webkit/support/test_shell_request_context.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M webkit/support/test_webkit_platform_support.h View 1 2 5 chunks +1 line, -24 lines 0 comments Download
M webkit/support/test_webkit_platform_support.cc View 1 2 3 8 chunks +4 lines, -62 lines 0 comments Download
M webkit/support/webkit_support.h View 1 2 3 chunks +0 lines, -24 lines 0 comments Download
M webkit/support/webkit_support.cc View 1 2 5 chunks +0 lines, -51 lines 0 comments Download
M webkit/support/webkit_support.gypi View 1 2 1 chunk +0 lines, -18 lines 0 comments Download

Messages

Total messages: 11 (0 generated)
scottmg
webkit_unit_tests seems to work fine at least...
7 years, 6 months ago (2013-06-20 04:51:19 UTC) #1
jamesr
lgtm https://codereview.chromium.org/17476002/diff/1/webkit/support/webkit_support.cc File webkit/support/webkit_support.cc (right): https://codereview.chromium.org/17476002/diff/1/webkit/support/webkit_support.cc#newcode770 webkit/support/webkit_support.cc:770: LOG(ERROR) << "XXX:" << accept; hmm, NOTREACHED() << ...
7 years, 6 months ago (2013-06-20 04:58:00 UTC) #2
scottmg
turns out they can all go now, woot. https://codereview.chromium.org/17476002/diff/1/webkit/support/webkit_support.cc File webkit/support/webkit_support.cc (right): https://codereview.chromium.org/17476002/diff/1/webkit/support/webkit_support.cc#newcode770 webkit/support/webkit_support.cc:770: LOG(ERROR) ...
7 years, 6 months ago (2013-06-20 05:04:32 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/scottmg@chromium.org/17476002/5001
7 years, 6 months ago (2013-06-24 14:33:18 UTC) #4
commit-bot: I haz the power
Failed to apply patch for webkit/support/test_webkit_platform_support.h: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
7 years, 6 months ago (2013-06-24 14:33:23 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/scottmg@chromium.org/17476002/16001
7 years, 6 months ago (2013-06-24 17:01:23 UTC) #6
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
7 years, 6 months ago (2013-06-24 17:21:54 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/scottmg@chromium.org/17476002/28002
7 years, 6 months ago (2013-06-24 17:25:16 UTC) #8
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
7 years, 6 months ago (2013-06-24 17:53:18 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/scottmg@chromium.org/17476002/26002
7 years, 6 months ago (2013-06-24 18:02:43 UTC) #10
commit-bot: I haz the power
7 years, 6 months ago (2013-06-24 21:47:12 UTC) #11
Message was sent while issue was closed.
Change committed as 208265

Powered by Google App Engine
This is Rietveld 408576698