Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2011 Google Inc. All rights reserved. | 2 * Copyright (C) 2011 Google Inc. All rights reserved. |
| 3 * | 3 * |
| 4 * Redistribution and use in source and binary forms, with or without | 4 * Redistribution and use in source and binary forms, with or without |
| 5 * modification, are permitted provided that the following conditions are | 5 * modification, are permitted provided that the following conditions are |
| 6 * met: | 6 * met: |
| 7 * | 7 * |
| 8 * * Redistributions of source code must retain the above copyright | 8 * * Redistributions of source code must retain the above copyright |
| 9 * notice, this list of conditions and the following disclaimer. | 9 * notice, this list of conditions and the following disclaimer. |
| 10 * * Redistributions in binary form must reproduce the above | 10 * * Redistributions in binary form must reproduce the above |
| (...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 63 return stringBuilder.toString(); | 63 return stringBuilder.toString(); |
| 64 } | 64 } |
| 65 | 65 |
| 66 MHTMLArchive::MHTMLArchive() | 66 MHTMLArchive::MHTMLArchive() |
| 67 { | 67 { |
| 68 } | 68 } |
| 69 | 69 |
| 70 MHTMLArchive* MHTMLArchive::create(const KURL& url, PassRefPtr<SharedBuffer> dat a) | 70 MHTMLArchive* MHTMLArchive::create(const KURL& url, PassRefPtr<SharedBuffer> dat a) |
| 71 { | 71 { |
| 72 // MHTML pages can only be loaded from local URLs and http/https URLs. | 72 // MHTML pages can only be loaded from local URLs and http/https URLs. |
| 73 // The latter is now allowed due to full sandboxing enforcement on MHTML pag es. | 73 // The latter is now allowed due to full sandboxing enforcement on MHTML pag es. |
|
jianli
2016/08/15 23:12:02
Please also update comment.
Vivian
2016/08/16 21:00:44
Done.
| |
| 74 if (!SchemeRegistry::shouldTreatURLSchemeAsLocal(url.protocol()) && !url.pro tocolIsInHTTPFamily()) | 74 if (!SchemeRegistry::shouldTreatURLSchemeAsLocal(url.protocol()) && !url.pro tocolIsInHTTPFamily() && !url.protocolIsContent()) |
|
jianli
2016/08/15 23:12:02
The check for content protocol should only be for
Vivian
2016/08/16 21:00:44
Done.
| |
| 75 return nullptr; | 75 return nullptr; |
| 76 | 76 |
| 77 MHTMLParser parser(data); | 77 MHTMLParser parser(data); |
| 78 HeapVector<Member<ArchiveResource>> resources = parser.parseArchive(); | 78 HeapVector<Member<ArchiveResource>> resources = parser.parseArchive(); |
| 79 if (resources.isEmpty()) | 79 if (resources.isEmpty()) |
| 80 return nullptr; // Invalid MHTML file. | 80 return nullptr; // Invalid MHTML file. |
| 81 | 81 |
| 82 MHTMLArchive* archive = new MHTMLArchive; | 82 MHTMLArchive* archive = new MHTMLArchive; |
| 83 // The first document suitable resource is the main resource of the top fram e. | 83 // The first document suitable resource is the main resource of the top fram e. |
| 84 for (size_t i = 0; i < resources.size(); ++i) { | 84 for (size_t i = 0; i < resources.size(); ++i) { |
| (...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 238 return m_subresources.get(url.getString()); | 238 return m_subresources.get(url.getString()); |
| 239 } | 239 } |
| 240 | 240 |
| 241 DEFINE_TRACE(MHTMLArchive) | 241 DEFINE_TRACE(MHTMLArchive) |
| 242 { | 242 { |
| 243 visitor->trace(m_mainResource); | 243 visitor->trace(m_mainResource); |
| 244 visitor->trace(m_subresources); | 244 visitor->trace(m_subresources); |
| 245 } | 245 } |
| 246 | 246 |
| 247 } // namespace blink | 247 } // namespace blink |
| OLD | NEW |