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

Side by Side Diff: third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.cpp

Issue 2842253002: Move ReportLocalLoadFailed to ExecutionContext (Closed)
Patch Set: Created 3 years, 7 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, 2007, 2009 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2009 Apple Inc. All rights reserved.
3 * 3 *
4 * This library is free software; you can redistribute it and/or 4 * This library is free software; you can redistribute it and/or
5 * modify it under the terms of the GNU Library General Public 5 * modify it under the terms of the GNU Library General Public
6 * License as published by the Free Software Foundation; either 6 * License as published by the Free Software Foundation; either
7 * version 2 of the License, or (at your option) any later version. 7 * version 2 of the License, or (at your option) any later version.
8 * 8 *
9 * This library is distributed in the hope that it will be useful, 9 * This library is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 326 matching lines...) Expand 10 before | Expand all | Expand 10 after
337 UpdateContainerPolicy(); 337 UpdateContainerPolicy();
338 338
339 LocalFrame* parent_frame = GetDocument().GetFrame(); 339 LocalFrame* parent_frame = GetDocument().GetFrame();
340 if (ContentFrame()) { 340 if (ContentFrame()) {
341 ContentFrame()->Navigate(GetDocument(), url, replace_current_item, 341 ContentFrame()->Navigate(GetDocument(), url, replace_current_item,
342 UserGestureStatus::kNone); 342 UserGestureStatus::kNone);
343 return true; 343 return true;
344 } 344 }
345 345
346 if (!GetDocument().GetSecurityOrigin()->CanDisplay(url)) { 346 if (!GetDocument().GetSecurityOrigin()->CanDisplay(url)) {
347 FrameLoader::ReportLocalLoadFailed(parent_frame, url.GetString()); 347 GetDocument().ReportLocalLoadFailed(url.GetString());
348 return false; 348 return false;
349 } 349 }
350 350
351 if (!SubframeLoadingDisabler::CanLoadFrame(*this)) 351 if (!SubframeLoadingDisabler::CanLoadFrame(*this))
352 return false; 352 return false;
353 353
354 if (GetDocument().GetFrame()->GetPage()->SubframeCount() >= 354 if (GetDocument().GetFrame()->GetPage()->SubframeCount() >=
355 Page::kMaxNumberOfFrames) 355 Page::kMaxNumberOfFrames)
356 return false; 356 return false;
357 357
(...skipping 11 matching lines...) Expand all
369 } 369 }
370 370
371 DEFINE_TRACE(HTMLFrameOwnerElement) { 371 DEFINE_TRACE(HTMLFrameOwnerElement) {
372 visitor->Trace(content_frame_); 372 visitor->Trace(content_frame_);
373 visitor->Trace(widget_); 373 visitor->Trace(widget_);
374 HTMLElement::Trace(visitor); 374 HTMLElement::Trace(visitor);
375 FrameOwner::Trace(visitor); 375 FrameOwner::Trace(visitor);
376 } 376 }
377 377
378 } // namespace blink 378 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698