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

Side by Side Diff: Source/WebCore/dom/Document.cpp

Issue 7980042: Merge 95471 - [Chromium] Crash after magic iframe transfer for Pepper/NaCl plugins. (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/874/
Patch Set: Created 9 years, 3 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) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org) 5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights reserved. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights reserved.
7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/) 7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/)
8 * Copyright (C) 2008, 2009 Google Inc. All rights reserved. 8 * Copyright (C) 2008, 2009 Google Inc. All rights reserved.
9 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies) 9 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
10 * 10 *
(...skipping 922 matching lines...) Expand 10 before | Expand all | Expand 10 after
933 attr->setSpecified(true); 933 attr->setSpecified(true);
934 break; 934 break;
935 } 935 }
936 default: 936 default:
937 if (source->hasTagName(iframeTag)) { 937 if (source->hasTagName(iframeTag)) {
938 HTMLIFrameElement* iframe = static_cast<HTMLIFrameElement*>(source.g et()); 938 HTMLIFrameElement* iframe = static_cast<HTMLIFrameElement*>(source.g et());
939 if (frame() && frame()->tree()->isDescendantOf(iframe->contentFrame( ))) { 939 if (frame() && frame()->tree()->isDescendantOf(iframe->contentFrame( ))) {
940 ec = HIERARCHY_REQUEST_ERR; 940 ec = HIERARCHY_REQUEST_ERR;
941 return 0; 941 return 0;
942 } 942 }
943 iframe->setRemainsAliveOnRemovalFromTree(attached() && source->attac hed()); 943 iframe->setRemainsAliveOnRemovalFromTree(attached() && source->attac hed() && iframe->canRemainAliveOnRemovalFromTree());
944 } 944 }
945 945
946 if (source->parentNode()) 946 if (source->parentNode())
947 source->parentNode()->removeChild(source.get(), ec); 947 source->parentNode()->removeChild(source.get(), ec);
948 } 948 }
949 949
950 source->setTreeScopeRecursively(this); 950 source->setTreeScopeRecursively(this);
951 951
952 return source; 952 return source;
953 } 953 }
(...skipping 4205 matching lines...) Expand 10 before | Expand all | Expand 10 after
5159 if (!loader) 5159 if (!loader)
5160 return 0; 5160 return 0;
5161 5161
5162 if (m_frame->document() != this) 5162 if (m_frame->document() != this)
5163 return 0; 5163 return 0;
5164 5164
5165 return loader; 5165 return loader;
5166 } 5166 }
5167 5167
5168 } // namespace WebCore 5168 } // namespace WebCore
OLDNEW
« no previous file with comments | « LayoutTests/fast/frames/resources/iframe-reparenting-embed-iframe.html ('k') | Source/WebCore/html/HTMLFrameElementBase.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698