|
Revert of Set proper referrer for css resource fetching ( https://codereview.chromium.org/239353007/)
Reason for revert:
Caused regression
Original issue's description:
> Set proper referrer for css resource fetching
>
> Fetching css resources are initiated as follows:
> - Create FetchRequest
> - Call ResourceFetcher::fetchXXX()
>
> git-grep shows me that CSSImageValue, CSSImageSetValue and
> CSSFontFaceSrcValue are all of the resource fetch initiators
> which live in css/. I think that setting referrer in these
> classes should cover all css resource fetching.
>
> Since resource fetching is delayed until the resource is actually
> needed, we need to hold the referrer somewhere.
> This CL modifies the css parser to set referrer to corresponding
> CSSValue (CSSImageValue and CSSFontFaceSrcValue) when it sees url(...).
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=172433
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175004
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+12 lines, -268 lines) |
Patch |
 |
D |
LayoutTests/http/tests/css/css-resources-referrer.html
|
View
|
|
1 chunk |
+0 lines, -48 lines |
0 comments
|
Download
|
 |
D |
LayoutTests/http/tests/css/css-resources-referrer-expected.html
|
View
|
|
1 chunk |
+0 lines, -34 lines |
0 comments
|
Download
|
 |
D |
LayoutTests/http/tests/css/css-resources-referrer-srcdoc.html
|
View
|
|
1 chunk |
+0 lines, -36 lines |
0 comments
|
Download
|
 |
D |
LayoutTests/http/tests/css/css-resources-referrer-srcdoc-expected.html
|
View
|
|
1 chunk |
+0 lines, -23 lines |
0 comments
|
Download
|
 |
D |
LayoutTests/http/tests/css/resources/css-resources-referrer.css
|
View
|
|
1 chunk |
+0 lines, -16 lines |
0 comments
|
Download
|
 |
D |
LayoutTests/http/tests/css/resources/css-resources-referrer-import.css
|
View
|
|
1 chunk |
+0 lines, -14 lines |
0 comments
|
Download
|
 |
D |
LayoutTests/http/tests/css/resources/referrer-check.php
|
View
|
|
1 chunk |
+0 lines, -52 lines |
0 comments
|
Download
|
 |
M |
Source/core/css/CSSCursorImageValue.cpp
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
 |
M |
Source/core/css/CSSFontFaceSrcValue.h
|
View
|
|
2 chunks |
+0 lines, -2 lines |
0 comments
|
Download
|
 |
M |
Source/core/css/CSSFontFaceSrcValue.cpp
|
View
|
|
1 chunk |
+0 lines, -3 lines |
0 comments
|
Download
|
 |
M |
Source/core/css/CSSImageSetValue.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
 |
M |
Source/core/css/CSSImageSetValue.cpp
|
View
|
|
3 chunks |
+2 lines, -5 lines |
0 comments
|
Download
|
 |
M |
Source/core/css/CSSImageValue.h
|
View
|
|
2 chunks |
+0 lines, -4 lines |
0 comments
|
Download
|
 |
M |
Source/core/css/CSSImageValue.cpp
|
View
|
|
1 chunk |
+0 lines, -2 lines |
0 comments
|
Download
|
 |
M |
Source/core/css/parser/CSSPropertyParser.h
|
View
|
|
1 chunk |
+0 lines, -2 lines |
0 comments
|
Download
|
 |
M |
Source/core/css/parser/CSSPropertyParser.cpp
|
View
|
|
8 chunks |
+6 lines, -14 lines |
0 comments
|
Download
|
 |
M |
Source/core/html/HTMLBodyElement.cpp
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
 |
M |
Source/core/html/HTMLTableElement.cpp
|
View
|
|
1 chunk |
+2 lines, -5 lines |
0 comments
|
Download
|
 |
M |
Source/core/html/HTMLTablePartElement.cpp
|
View
|
|
1 chunk |
+2 lines, -5 lines |
0 comments
|
Download
|
Total messages: 14 (0 generated)
|