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

Side by Side Diff: third_party/WebKit/Source/core/dom/Document.cpp

Issue 2507023002: Support script modify iframe scrolling, marginWidth and marginHeight attr (Closed)
Patch Set: alexmos@ comment#20 addressed. but layouttest still not working with flag Created 4 years 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, 2012 Apple Inc. All 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All
7 * rights reserved. 7 * rights reserved.
8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. 8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
9 * (http://www.torchmobile.com/) 9 * (http://www.torchmobile.com/)
10 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved. 10 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
(...skipping 4292 matching lines...) Expand 10 before | Expand all | Expand 10 after
4303 } 4303 }
4304 4304
4305 HTMLFrameOwnerElement* Document::localOwner() const { 4305 HTMLFrameOwnerElement* Document::localOwner() const {
4306 if (!frame()) 4306 if (!frame())
4307 return 0; 4307 return 0;
4308 // FIXME: This probably breaks the attempts to layout after a load is finished 4308 // FIXME: This probably breaks the attempts to layout after a load is finished
4309 // in implicitClose(), and probably tons of other things... 4309 // in implicitClose(), and probably tons of other things...
4310 return frame()->deprecatedLocalOwner(); 4310 return frame()->deprecatedLocalOwner();
4311 } 4311 }
4312 4312
4313 void Document::willChangeFrameOwnerProperties(int marginWidth,
4314 int marginHeight,
4315 ScrollbarMode scrollingMode) {
4316 DCHECK(frame() && frame()->owner());
4317 FrameOwner* owner = frame()->owner();
4318
4319 if (marginWidth != owner->marginWidth())
4320 body()->setIntegralAttribute(marginwidthAttr, marginWidth);
4321 if (marginHeight != owner->marginHeight())
4322 body()->setIntegralAttribute(marginheightAttr, marginHeight);
4323 if (scrollingMode != owner->scrollingMode())
4324 view()->setNeedsLayout();
4325 }
4326
4313 bool Document::isInInvisibleSubframe() const { 4327 bool Document::isInInvisibleSubframe() const {
4314 if (!localOwner()) 4328 if (!localOwner())
4315 return false; // this is a local root element 4329 return false; // this is a local root element
4316 4330
4317 // TODO(bokan): This looks like it doesn't work in OOPIF. 4331 // TODO(bokan): This looks like it doesn't work in OOPIF.
4318 DCHECK(frame()); 4332 DCHECK(frame());
4319 return frame()->ownerLayoutItem().isNull(); 4333 return frame()->ownerLayoutItem().isNull();
4320 } 4334 }
4321 4335
4322 String Document::cookie(ExceptionState& exceptionState) const { 4336 String Document::cookie(ExceptionState& exceptionState) const {
(...skipping 2117 matching lines...) Expand 10 before | Expand all | Expand 10 after
6440 } 6454 }
6441 6455
6442 void showLiveDocumentInstances() { 6456 void showLiveDocumentInstances() {
6443 WeakDocumentSet& set = liveDocumentSet(); 6457 WeakDocumentSet& set = liveDocumentSet();
6444 fprintf(stderr, "There are %u documents currently alive:\n", set.size()); 6458 fprintf(stderr, "There are %u documents currently alive:\n", set.size());
6445 for (Document* document : set) 6459 for (Document* document : set)
6446 fprintf(stderr, "- Document %p URL: %s\n", document, 6460 fprintf(stderr, "- Document %p URL: %s\n", document,
6447 document->url().getString().utf8().data()); 6461 document->url().getString().utf8().data());
6448 } 6462 }
6449 #endif 6463 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698