angler-fishThe Vulnerability History Project

Revert 162155 &quotThis review merges the two existing page serializ...&quot

      Change r162155 broke the world even though it was landed using the CQ.

> This review merges the two existing page serializers, WebPageSerializerImpl and
> PageSerializer, into one, PageSerializer. In addition to this it moves all
> the old tests from WebPageNewSerializerTest and WebPageSerializerTest to the
> PageSerializerTest structure and splits out one test for MHTML into a new
> MHTMLTest file.
> 
> Saving as 'Webpage, Complete', 'Webpage, HTML Only' and as MHTML when the
> 'Save Page as MHTML' flag is enabled now uses the same code, and should thus
> have the same feature set. Meaning that both modes now should be a bit better.
> 
> Detailed list of changes:
> 
> - PageSerializerTest: Prepare for more DTD test
> - PageSerializerTest: Remove now unneccesary input image test
> - PageSerializerTest: Remove unused WebPageSerializer/Impl code
> - PageSerializerTest: Move data URI morph test
> - PageSerializerTest: Move data URI test
> - PageSerializerTest: Move namespace test
> - PageSerializerTest: Move SVG Image test
> - MHTMLTest: Move MHTML specific test to own test file
> - PageSerializerTest: Delete duplicate XML header test
> - PageSerializerTest: Move blank frame test
> - PageSerializerTest: Move CSS test
> - PageSerializerTest: Add frameset/frame test
> - PageSerializerTest: Move old iframe test
> - PageSerializerTest: Move old elements test
> - Use PageSerizer for saving web pages
> - PageSerializerTest: Test for rewriting links
> - PageSerializer: Add rewrite link accumulator
> - PageSerializer: Serialize images in iframes/frames src
> - PageSerializer: XHTML fix for meta tags
> - PageSerializer: Add presentation CSS
> - PageSerializer: Rename out parameter
> 
> BUG=
> R=abarth@chromium.org
> 
> Review URL: https://codereview.chromium.org/68613003

TBR=tiger@opera.com

Review URL: https://codereview.chromium.org/73673003

git-svn-id: svn://svn.chromium.org/blink/trunk@162156 bb
    
commit 7f0126ff011142c8619b10a6e64d04d1745c503a
+1 -2
+1 -1
+117 -24
+7 -1
+15 -181
-527
-192
+169
+276 -14
+1 -1
-440
-193
-1
-1
-14
-50
-28
+3 -5
+10
+3 -3
+2 -2
+45
+3
+1
+3 -4
-2
-2
+14
+3
+3
+7
+5
+21
-9
-9
-25
+3
-9
-6
+13
+9
+9
-12
+1 -6
+5 -23
+2 -2
expand_less