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

Side by Side Diff: third_party/WebKit/Source/core/svg/graphics/SVGImage.cpp

Issue 2762423003: Made Frame constructors (and subclasses) take Page& instead of Page* (Closed)
Patch Set: Rebase Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006 Eric Seidel <eric@webkit.org> 2 * Copyright (C) 2006 Eric Seidel <eric@webkit.org>
3 * Copyright (C) 2008, 2009 Apple Inc. All rights reserved. 3 * Copyright (C) 2008, 2009 Apple Inc. All rights reserved.
4 * Copyright (C) Research In Motion Limited 2011. All rights reserved. 4 * Copyright (C) Research In Motion Limited 2011. All rights reserved.
5 * 5 *
6 * Redistribution and use in source and binary forms, with or without 6 * Redistribution and use in source and binary forms, with or without
7 * modification, are permitted provided that the following conditions 7 * modification, are permitted provided that the following conditions
8 * are met: 8 * are met:
9 * 1. Redistributions of source code must retain the above copyright 9 * 1. Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 657 matching lines...) Expand 10 before | Expand all | Expand 10 after
668 page->settings().setDefaultFontSize( 668 page->settings().setDefaultFontSize(
669 defaultSettings.getDefaultFontSize()); 669 defaultSettings.getDefaultFontSize());
670 page->settings().setDefaultFixedFontSize( 670 page->settings().setDefaultFixedFontSize(
671 defaultSettings.getDefaultFixedFontSize()); 671 defaultSettings.getDefaultFixedFontSize());
672 } 672 }
673 } 673 }
674 674
675 LocalFrame* frame = nullptr; 675 LocalFrame* frame = nullptr;
676 { 676 {
677 TRACE_EVENT0("blink", "SVGImage::dataChanged::createFrame"); 677 TRACE_EVENT0("blink", "SVGImage::dataChanged::createFrame");
678 frame = LocalFrame::create(&dummyLocalFrameClient, page, 0); 678 frame = LocalFrame::create(&dummyLocalFrameClient, *page, 0);
679 frame->setView(FrameView::create(*frame)); 679 frame->setView(FrameView::create(*frame));
680 frame->init(); 680 frame->init();
681 } 681 }
682 682
683 FrameLoader& loader = frame->loader(); 683 FrameLoader& loader = frame->loader();
684 loader.forceSandboxFlags(SandboxAll); 684 loader.forceSandboxFlags(SandboxAll);
685 685
686 frame->view()->setScrollbarsSuppressed(true); 686 frame->view()->setScrollbarsSuppressed(true);
687 // SVG Images will always synthesize a viewBox, if it's not available, and 687 // SVG Images will always synthesize a viewBox, if it's not available, and
688 // thus never see scrollbars. 688 // thus never see scrollbars.
(...skipping 15 matching lines...) Expand all
704 } 704 }
705 705
706 return m_page ? SizeAvailable : SizeUnavailable; 706 return m_page ? SizeAvailable : SizeUnavailable;
707 } 707 }
708 708
709 String SVGImage::filenameExtension() const { 709 String SVGImage::filenameExtension() const {
710 return "svg"; 710 return "svg";
711 } 711 }
712 712
713 } // namespace blink 713 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698