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

Issue 220010: Store non-standard HTTP headers in history. (WebKit side)

Created:
11 years, 3 months ago by Marshall Greenblatt
Modified:
11 years, 3 months ago
CC:
chromium-reviews_googlegroups.com
Base URL:
http://svn.webkit.org/repository/webkit/trunk/WebCore/
Visibility:
Public.

Description

We currently have the ability to set extra HTTP header fields in WebURLRequest. However, the extra HTTP header fields are not stored in WebHistoryItem and are therefore lost after navigation. Non-standard HTTP headers, usually prefixed with 'X-', are used by a number of server-based applications and represent a safe subset of extra HTTP headers that can be stored in the history. This patch works in combination with the patch for the below WebKit issue to support storage and recall of non-standard HTTP headers. https://bugs.webkit.org/show_bug.cgi?id=26994 The chromium side of this patch is available here: http://codereview.chromium.org/225012

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+74 lines, -1 line) Patch
M history/HistoryItem.h View 1 2 3 5 chunks +8 lines, -0 lines 0 comments Download
M history/HistoryItem.cpp View 1 2 3 1 chunk +59 lines, -1 line 0 comments Download
M loader/FrameLoader.cpp View 1 2 3 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Marshall Greenblatt
11 years, 3 months ago (2009-09-23 17:41:22 UTC) #1
Please review this patch.

Powered by Google App Engine
This is Rietveld 408576698