12025 2006-12-28 21:08:00 -0800 WebArchive fails to save background images in <tr> tags 2015-08-04 09:41:36 -0700 1 1 1 Unclassified WebKit WebKit Misc. 420+ Mac OS X 10.4 REOPENED P2 Normal --- 1 ddkilzer webkit-unassigned bburg beidson oldest_to_newest 39101 0 ddkilzer 2006-12-28 21:08:00 -0800 When a .webarchive file is saved, any background images in <tr> tags are not serialized to the output file. 39102 1 12098 ddkilzer 2006-12-28 21:11:01 -0800 Created attachment 12098 WebArchive layout test (as patch) WebArchive layout test formatted as a patch. 76212 2 beidson 2008-04-02 23:17:04 -0700 This one is going to be very easy to fix with the new archive setup in WebCore However, I have a question... I find no evidence that <tr>s can officially have background images, standards-wise. Therefore, I'm guessing this is some Mozilla or IE compatibility thing we added. Looking at the HTMLTablePart code, it appears *any* table part can have a background image. Is that correct? If so, shouldn't this patch make sure all table parts get their background images saved? 76260 3 ddkilzer 2008-04-03 13:25:55 -0700 (In reply to comment #2) > This one is going to be very easy to fix with the new archive setup in WebCore > However, I have a question... I find no evidence that <tr>s can officially > have background images, standards-wise. Therefore, I'm guessing this is some > Mozilla or IE compatibility thing we added. > > Looking at the HTMLTablePart code, it appears *any* table part can have a > background image. Is that correct? > > If so, shouldn't this patch make sure all table parts get their background > images saved? At the time, I think I only tested <tr> tags. If HTMLTablePart supports background images on all parts, then we should archive them. 76273 4 beidson 2008-04-03 15:00:44 -0700 A few further points of discussion: -The background attribute isn't officially supported for anything but <body>, I think, so its weird and nonstandard that we support it for any table parts -The background attribute for HTMLTablePart is fed directly into inline style information and handled in a very CSSey fashion. -Therefore, while we could support archiving the resource pointed to by the background attribute for table parts specifically, there's a broader-reaching problem of archiving resources referred to by all inline style. Not saying I don't plan to fix this specific bug, but... the more accurate we get with esoteric subresources (like background images of <tr>s), the more apparent it is that we're ignoring a large class of subresources 1055545 5 burg 2014-12-14 19:59:28 -0800 WAR export isn't supported these days. 1055546 6 burg 2014-12-14 20:01:11 -0800 Oops, thought this was filed as an inspector bug. 12098 2006-12-28 21:11:01 -0800 2010-06-10 15:35:47 -0700 WebArchive layout test (as patch) bug-12025-layout-test.diff text/plain 5087 ddkilzer SW5kZXg6IExheW91dFRlc3RzL3dlYmFyY2hpdmUvdGVzdC10ci1iYWNrZ3JvdW5kLWV4cGVjdGVk LnR4dAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy93ZWJhcmNoaXZlL3Rlc3QtdHItYmFja2dy b3VuZC1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy93ZWJhcmNoaXZl L3Rlc3QtdHItYmFja2dyb3VuZC1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEs OTMgQEAKKzw/eG1sIHZlcnNpb249IjEuMCIgZW5jb2Rpbmc9IlVURi04Ij8+Cis8IURPQ1RZUEUg cGxpc3QgUFVCTElDICItLy9BcHBsZSBDb21wdXRlci8vRFREIFBMSVNUIDEuMC8vRU4iICJodHRw Oi8vd3d3LmFwcGxlLmNvbS9EVERzL1Byb3BlcnR5TGlzdC0xLjAuZHRkIj4KKzxwbGlzdCB2ZXJz aW9uPSIxLjAiPgorPGRpY3Q+CisJPGtleT5XZWJNYWluUmVzb3VyY2U8L2tleT4KKwk8ZGljdD4K KwkJPGtleT5XZWJSZXNvdXJjZURhdGE8L2tleT4KKwkJPGRhdGE+CisJCVBHaDBiV3crUEdobFlX UStDanh6WTNKcGNIUStDaUFnSUNCcFppQW9kMmx1Wkc5M0xteGhlVzkxZEZSbAorCQljM1JEYjI1 MGNtOXNiR1Z5S1FvZ0lDQWdJQ0FnSUd4aGVXOTFkRlJsYzNSRGIyNTBjbTlzYkdWeUxtUjEKKwkJ YlhCQmMxZGxZa0Z5WTJocGRtVW9LVHNLUEM5elkzSnBjSFErQ2p3dmFHVmhaRDQ4WW05a2VUNEtJ Q0E4CisJCWRHRmliR1UrQ2lBZ0lDQThkR0p2WkhrK1BIUnlJR0poWTJ0bmNtOTFibVE5SW5KbGMy OTFjbU5sY3k5aAorCQljSEJzWlM1bmFXWWlQangwWkNCemRIbHNaVDBpZDJsa2RHZzZJRFV5Y0hn N0lHaGxhV2RvZERvZ05qUncKKwkJZURzaVBnb2dJQ0FnSUNBOEwzUmtQand2ZEhJK1BDOTBZbTlr ZVQ0OEwzUmhZbXhsUGdvS0Nqd3ZZbTlrCisJCWVUNDhMMmgwYld3KworCQk8L2RhdGE+CisJCTxr ZXk+V2ViUmVzb3VyY2VGcmFtZU5hbWU8L2tleT4KKwkJPHN0cmluZz48L3N0cmluZz4KKwkJPGtl eT5XZWJSZXNvdXJjZU1JTUVUeXBlPC9rZXk+CisJCTxzdHJpbmc+dGV4dC9odG1sPC9zdHJpbmc+ CisJCTxrZXk+V2ViUmVzb3VyY2VUZXh0RW5jb2RpbmdOYW1lPC9rZXk+CisJCTxzdHJpbmc+VVRG LTg8L3N0cmluZz4KKwkJPGtleT5XZWJSZXNvdXJjZVVSTDwva2V5PgorCQk8c3RyaW5nPmZpbGU6 Ly8vTGF5b3V0VGVzdHMvd2ViYXJjaGl2ZS90ZXN0LXRyLWJhY2tncm91bmQuaHRtbDwvc3RyaW5n PgorCTwvZGljdD4KKwk8a2V5PldlYlN1YnJlc291cmNlczwva2V5PgorCTxhcnJheT4KKwkJPGRp Y3Q+CisJCQk8a2V5PldlYlJlc291cmNlRGF0YTwva2V5PgorCQkJPGRhdGE+CisJCQlSMGxHT0Rs aE5BQkFBTVFmQUdKaVl2THo5SW1LaXR2YjIvMzkvY25KeWFTa3BPUGo0OVRVCisJCQkxTUhCd2JX MXRlcnE3THU3dS9YMTl1WG01KzN1NytmbzZlRGc0ZS93OHQ3ZTMvbjUrUmNYCisJCQlGODdPei9m Mzk4Zkh4OWZYMTYrdnI5SFIwZkR3OFBqNCtNek16QUFBQUNINUJBRUFBQjhBCisJCQlMQUFBQUFB MEFFQUFRQVgvNENlT1pHbWU1aUZVbFlDK2NDd0tEN0ZBeHhFbEFpVERuQVRBCisJCQljQ0VZajhh QUJpQklEQ0lialFDZ3FCRW9qVUNqUVNFRXBvWFRBVUNRTEI0UGlab1RhRy9mCisJCQlqY3VGZ2ox a0JybklaRExvRHpJSWdSc1RBZ1ltQms4VEVRNE9FR2RwYkJRTERBVVFFQTU1CisJCQllMzJBR3hz V0hoNEZCUmdKQ1FnRkFoUW9Dd0lEbGdzU0Z4eUZPbnQ4Zm9BSW5wK2pwUWtNCisJCQl3QXJDR3NR YUNnMEtCZ0VvRHdBQUhvSFJncm9XMWFFWXBMN0EyOExEeGNVR0dnYWxoaThJCisJCQlBQWdZb092 cm41L3NvTDJsd2NUaEVCUUFBei83SnhZQVhWY0NQTmpnZ3AvQkV3SVlPSE5tCisJCQl3TUhCRXJN U0VNZ0NSNDdGQzNFd3dubHpvY09GQnhNeTNOSGtwODhFQndVLy8vZ2pjS0NSCisJCQlvMGNQT0Vo d002ZkRnVUdXTXVIS3RRdlVCZ0JoVGhoUTRDQkNTd2hvSkRUb1FLeGxqbHE0CisJCQlCUEhDTm8r YmdnWEVYbndaTU5NQXVwSi9ja205UnZVWGcyN2R2cWxsa0VGQUR4TVFBS2lyCisJCQlaczNEdTFD aVJwWDFaZmFzTjNFR0Fsc1lJSUFEdjNNQ0RuQm93S0VBUXdTR1NSekFNRVZBCisJCQlCSXdUbUNC NHlIa0VRUU1BQ1FEWTBMbXpnQWxYWkYwYWM2RDBRd1VKT0F4QWtNQXJnQXl1CisJCQlUM2dRNXpZ d0JoOGpCbWhnb0c5RUJnWUczQmJTUUhwZkJBRUtqSFNZVHQzanhRNEJIM0RCCisJCQkyRWFMbkFZ TElnQzZFeUd1QmhNRHlEU0kyWDNqbHU4VmF5NzQ4MFRQclUyNTBwOG4wU3JBCisJCQlTelF4c1RH VEZ1OWhGNEI0NVdFU2dmOGUrRVd6eTAvTmxUQk9lRGswQXROTWNWQVFRQ2NICisJCQk1TFRnTFNK SjljNG9vUWl3akZBSTdHR1VoV2tFMEFGV3YrR1FCNE4vNk9KSlBObk1ZMEVBCisJCQlBQ3lBZ21N U0xJS1VCQnhRZ0VvR1JYMElJazhqNWhpTU53eFFrQkFLREFoZ3hnTWRlQ0JYCisJCQlCR0NKRkdJ bnZNaGoxVi9mSk1Dam1TVkVBRUJjQUZpUVM0alM5SFNOTmsrbXBaWmFCbUF3CisJCQlXZ21aVWVO Sko0RFdKVXFPZEk2SloyRGpFSEtpQ1J3RUJnODhCYkNUbDE1TytsVVBvbzN1CisJCQk5NE1DQUVS QWlqcngrTlFIQnVJbzRFR05nQlJRaWdjUmRBRkFBcmw5RUFBYkJuV2dSYXlsCisJCQlPYkNFaVFR SUVDR3UrMmk1Z0JFYXhnVWNzREprMXNBVnRwNHhKYkl5NkprZEpoQkFCNjMvCisJCQlEQ3N0b09B R1hoVjNMUXhldWJXUVpkLytrQkJEdUYxN0FLY0FhRERBQXVFcDFHNEdEWmlBCisJCQlnQlJNWU9E QUFnZFlZQnNEK3pqZ0ZVQVVWR2VkRVJmOE5PNDlBUTJZa1g4NVpORHVDeW9VCisJCQljRVZHQmx1 bkVVVmJ1QkhIZEJ3Y01OdElma3dRRndZbFVNRERSQTV2OUIxOEZSVU1IbjIyCisJCQlnQlVXSVpw K1FFR2VCS1NoUm5jZXU1Y1JIYkxkb2ROTzBhU25nQWxWMnZDSUJENFBTT0I3CisJCQljMGhpZENi MzFUZ05EeWZFZGNFbEx6M0N3ZGd0MDdIQUhZMGNyYldON2lpUVVnbHVON0RpCisJCQlLMmVRM2NZ Y0Z3eXdRWWN5MGdnbm1MeDQ4Rk5yUWxsUTFGR3Z4SVFoQlErbzB3Z21VSVUxCisJCQlGbDdZYkVC WkJ5Z2dza2VGamlRVkFBVVFLRkRKLzJwNVJFVU5qbFN4MWRZRm1XT2doNUJpCisJCQlhemhjVGxo dk1wWmVkTUltMnE4a3VFMWg1ek5Sa014Sk05N0hKSTY1RDdOQkJ6MmVrTmtECisJCQlMWjBoUVFl RUpaQWtqYmFmUG1kVmRoS0RRQU1BRUM1aEFnOGc5WGx5ZU5ScytvMWtWZVdYCisJCQlBbmN5d0R3 RDlaS1FtU01CTERBRWwxM3lkQ1B1M0NQVG5SS2dQd1A0YUFSVnVvQmpFQkdXCisJCQk4Zml2U2Rr WUUvenVkQ2NQQ01BRHZVT1hOT0lFSnRRVkNpMEN4Rk42SGxDQ0tnRnFBNElJCisJCQkxRjBBNkw0 NmtRbFJnUmtEQmswZ2hWM1E1WVlReEFhaHR2R2tiOERRZ005NGdXUHVBb3E2CisJCQl4Q012R0tE VURsOFltT2lnSXdZeWpOU2pwcmlPUVVYUVVwZmlsd0FjOGdNaFpDQUJjOEhoCisJCQlOU0psZ1Rr T2dvSVVaeWtHK1VRVEZJTW9oQkVJS0dOOUJtQ2JRZ1JtQ3ErYXdBSXdnaDBiCisJCQl2TW8xQzZo TlpmSjB3QmRNd0cyOUtRQUp5L1d0RUFBQU8xQnZiRzl1ZFhNSworCQkJPC9kYXRhPgorCQkJPGtl eT5XZWJSZXNvdXJjZU1JTUVUeXBlPC9rZXk+CisJCQk8c3RyaW5nPmltYWdlL2dpZjwvc3RyaW5n PgorCQkJPGtleT5XZWJSZXNvdXJjZVJlc3BvbnNlPC9rZXk+CisJCQk8ZGF0YT4KKwkJCVluQnNh WE4wTUREVUFRSURCQVVHQndwWkpHRnlZMmhwZG1WeVdDUjJaWEp6YVc5dVZDUjAKKwkJCWIzQllK RzlpYW1WamRITmZFQTlPVTB0bGVXVmtRWEpqYUdsMlpYSVNBQUdHb05FSUNWOFEKKwkJCUUxZGxZ bEpsYzI5MWNtTmxVbVZ6Y0c5dWMyV0FBYWNMREJzaElpa3FWU1J1ZFd4czF3ME8KKwkJCUR4QVJF aE1VRlJZWEdCa2FVaVF5VWlRelVpUTBVaVF3VmlSamJHRnpjMUlrTlZJa01ZQUYKKwkJCWdBQVJC Y1FRQjRBR0kzL3YvLy8vLy8vL2dBTFRIQjBSRlI4Z1YwNVRMbUpoYzJWYlRsTXUKKwkJCWNtVnNZ WFJwZG1XQUFJQURnQVJmRUZCbWFXeGxPaTh2TDFWelpYSnpMMlJyYVd4NlpYSXYKKwkJCVVISnZh bVZqZEhNdlYyVmlTMmwwTDB4aGVXOTFkRlJsYzNSekwzZGxZbUZ5WTJocGRtVXYKKwkJCWNtVnpi M1Z5WTJWekwyRndjR3hsTG1kcFp0SWpKQ1VtV0NSamJHRnpjMlZ6V2lSamJHRnoKKwkJCWMyNWhi V1dqSmljb1ZVNVRWVkpNVmlWT1UxVlNURmhPVTA5aWFtVmpkRmxwYldGblpTOW4KKwkJCWFXYlNJ eVFyTEtJc0tGMU9VMVZTVEZKbGMzQnZibk5sQUFnQUVRQWJBQ1FBS1FBeUFFUUEKKwkJCVNRQk1B R0lBWkFCc0FISUFnUUNFQUljQWlnQ05BSlFBbHdDYUFKd0FuZ0NoQUtNQXBRQ3UKKwkJCUFMQUF0 d0MvQU1zQXpRRFBBTkVCSkFFcEFUSUJQUUZCQVVjQlRnRlhBV0VCWmdGcEFBQUEKKwkJCUFBQUFB Z0VBQUFBQUFBQUFMUUFBQUFBQUFBQUFBQUFBQUFBQUFYYz0KKwkJCTwvZGF0YT4KKwkJCTxrZXk+ V2ViUmVzb3VyY2VVUkw8L2tleT4KKwkJCTxzdHJpbmc+ZmlsZTovLy9MYXlvdXRUZXN0cy93ZWJh cmNoaXZlL3Jlc291cmNlcy9hcHBsZS5naWY8L3N0cmluZz4KKwkJPC9kaWN0PgorCTwvYXJyYXk+ Cis8L2RpY3Q+Cis8L3BsaXN0PgpJbmRleDogTGF5b3V0VGVzdHMvd2ViYXJjaGl2ZS90ZXN0LXRy LWJhY2tncm91bmQuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy93ZWJhcmNoaXZlL3Rl c3QtdHItYmFja2dyb3VuZC5odG1sCShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvd2ViYXJj aGl2ZS90ZXN0LXRyLWJhY2tncm91bmQuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwxNiBA QAorPGh0bWw+Cis8aGVhZD4KKzxzY3JpcHQ+CisgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29u dHJvbGxlcikKKyAgICAgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIuZHVtcEFzV2ViQXJjaGl2ZSgp OworPC9zY3JpcHQ+Cis8L2hlYWQ+Cis8Ym9keT4KKyAgPHRhYmxlPgorICAgIDx0ciBiYWNrZ3Jv dW5kPSJyZXNvdXJjZXMvYXBwbGUuZ2lmIj4KKyAgICAgIDx0ZCBzdHlsZT0id2lkdGg6IDUycHg7 IGhlaWdodDogNjRweDsiPgorICAgICAgPC90ZD4KKyAgICA8L3RyPgorICA8L3RhYmxlPgorPC9i b2R5PgorPC9odG1sPgo=