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

Issue 8680036: Move ResourceResponse struct into the Content API, since it's used in Chrome. While at it, I also... (Closed)

Created:
9 years, 1 month ago by jam
Modified:
9 years, 1 month ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews, cbentzel+watch_chromium.org, brettw-cc_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, rdsmith+dwatch_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Move ResourceResponse struct into the Content API, since it's used in Chrome. While at it, I also made the ResourceResponse wrapper just derive from ResourceResponseHead, so that the parameters are accessed directly, instead of having to go through a member variable. BUG=98716 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111471

Patch Set 1 #

Total comments: 2

Patch Set 2 : fix chromeos #

Patch Set 3 : add gypi changes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+233 lines, -267 lines) Patch
M chrome/browser/download/download_throttling_resource_handler.h View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/download/download_throttling_resource_handler.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/net/load_timing_observer.h View 2 chunks +5 lines, -3 lines 0 comments Download
M chrome/browser/net/load_timing_observer.cc View 3 chunks +7 lines, -6 lines 0 comments Download
M chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/renderer_host/offline_resource_handler.h View 1 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/renderer_host/offline_resource_handler.cc View 1 1 chunk +8 lines, -6 lines 0 comments Download
M chrome/browser/renderer_host/safe_browsing_resource_handler.h View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/renderer_host/safe_browsing_resource_handler.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M content/browser/debugger/devtools_netlog_observer.h View 2 chunks +6 lines, -3 lines 0 comments Download
M content/browser/debugger/devtools_netlog_observer.cc View 3 chunks +5 lines, -4 lines 0 comments Download
M content/browser/download/download_resource_handler.h View 1 chunk +2 lines, -2 lines 0 comments Download
M content/browser/download/download_resource_handler.cc View 5 chunks +13 lines, -11 lines 0 comments Download
M content/browser/download/save_file_resource_handler.h View 1 chunk +2 lines, -2 lines 0 comments Download
M content/browser/download/save_file_resource_handler.cc View 1 chunk +8 lines, -6 lines 0 comments Download
M content/browser/renderer_host/async_resource_handler.h View 1 chunk +2 lines, -2 lines 0 comments Download
M content/browser/renderer_host/async_resource_handler.cc View 4 chunks +11 lines, -9 lines 0 comments Download
M content/browser/renderer_host/buffered_resource_handler.h View 2 chunks +3 lines, -3 lines 0 comments Download
M content/browser/renderer_host/buffered_resource_handler.cc View 9 chunks +18 lines, -17 lines 0 comments Download
M content/browser/renderer_host/cross_site_resource_handler.h View 3 chunks +4 lines, -4 lines 0 comments Download
M content/browser/renderer_host/cross_site_resource_handler.cc View 4 chunks +11 lines, -10 lines 0 comments Download
M content/browser/renderer_host/redirect_to_file_resource_handler.h View 1 chunk +2 lines, -2 lines 0 comments Download
M content/browser/renderer_host/redirect_to_file_resource_handler.cc View 3 chunks +5 lines, -5 lines 0 comments Download
M content/browser/renderer_host/resource_dispatcher_host.cc View 4 chunks +16 lines, -17 lines 0 comments Download
M content/browser/renderer_host/resource_dispatcher_host_unittest.cc View 6 chunks +6 lines, -6 lines 0 comments Download
M content/browser/renderer_host/resource_handler.h View 2 chunks +8 lines, -5 lines 0 comments Download
M content/browser/renderer_host/sync_resource_handler.h View 3 chunks +4 lines, -4 lines 0 comments Download
M content/browser/renderer_host/sync_resource_handler.cc View 3 chunks +18 lines, -16 lines 0 comments Download
M content/browser/renderer_host/x509_user_cert_resource_handler.h View 1 chunk +2 lines, -2 lines 0 comments Download
M content/browser/renderer_host/x509_user_cert_resource_handler.cc View 2 chunks +10 lines, -8 lines 0 comments Download
M content/common/resource_dispatcher.h View 2 chunks +2 lines, -3 lines 0 comments Download
M content/common/resource_dispatcher.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M content/common/resource_dispatcher_unittest.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M content/common/resource_messages.h View 6 chunks +15 lines, -15 lines 0 comments Download
D content/common/resource_response.h View 1 chunk +0 lines, -47 lines 0 comments Download
D content/common/resource_response.cc View 1 chunk +0 lines, -11 lines 0 comments Download
M content/content_browser.gypi View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M content/content_common.gypi View 1 2 1 chunk +0 lines, -2 lines 0 comments Download
M content/public/browser/resource_dispatcher_host_delegate.h View 1 chunk +1 line, -1 line 0 comments Download
A + content/public/common/resource_response.h View 2 chunks +11 lines, -8 lines 0 comments Download
M content/test/render_view_fake_resources_test.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jam
9 years, 1 month ago (2011-11-24 00:00:35 UTC) #1
Dirk Pranke
lgtm, apart from the one question below (which I don't actually know the right answer ...
9 years, 1 month ago (2011-11-24 00:30:47 UTC) #2
jam
9 years, 1 month ago (2011-11-24 00:42:18 UTC) #3
http://codereview.chromium.org/8680036/diff/1/content/public/common/resource_...
File content/public/common/resource_response.h (right):

http://codereview.chromium.org/8680036/diff/1/content/public/common/resource_...
content/public/common/resource_response.h:45: ~ResourceResponse() {}
On 2011/11/24 00:30:47, Dirk Pranke wrote:
> I'm still a bit fuzzy on the rules for when we're allowed to inline
constructors
> and destructors ... here it looks like you're letting the compiler generate a
> public constructor, and you're declaring the destructor private so that it
can't
> be subclassed, right? Are you sure that both of these should be inlined, since
> they contain strings and GURLS?

The errors I get when there are template member variables etc. I did send this
through the clang bot which didn't complain

Powered by Google App Engine
This is Rietveld 408576698