Plumb document's strict mixed content checking for RemoteFrames

third_party/WebKit/Source/core/dom/Document.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) Research In Motion Limited 2010-2011. All rights reserved.
@@ skipping 429 matching lines @@
     , m_registrationContext(initializer.registrationContext(this))
     , m_elementDataCacheClearTimer(this, &Document::elementDataCacheClearTimerFired)
     , m_timeline(AnimationTimeline::create(this))
     , m_templateDocumentHost(nullptr)
     , m_didAssociateFormControlsTimer(this, &Document::didAssociateFormControlsTimerFired)
     , m_timers(timerTaskRunner()->adoptClone())
     , m_hasViewportUnits(false)
     , m_styleRecalcElementCounter(0)
     , m_parserSyncPolicy(AllowAsynchronousParsing)
     , m_nodeCount(0)
+    , m_strictMixedContentCheckingEnforced(false)
 {
     if (m_frame) {
         ASSERT(m_frame->page());
         provideContextFeaturesToDocumentFrom(*this, *m_frame->page());
 
         m_fetcher = m_frame->loader().documentLoader()->fetcher();
         FrameFetchContext::provideDocumentToContext(m_fetcher->context(), this);
     } else if (m_importsController) {
         m_fetcher = FrameFetchContext::createContextAndFetcher(nullptr);
         FrameFetchContext::provideDocumentToContext(m_fetcher->context(), this);
@@ skipping 5282 matching lines @@
     return Platform::current()->currentThread()->scheduler()->loadingTaskRunner();
 }
 
 WebTaskRunner* Document::timerTaskRunner() const
 {
     if (frame())
         return m_frame->frameScheduler()->timerTaskRunner();
     return Platform::current()->currentThread()->scheduler()->timerTaskRunner();
 }
 
+void Document::enforceStrictMixedContentChecking()
+{
+    m_strictMixedContentCheckingEnforced = true;
+    if (frame())
+        frame()->loader().client()->didChangeShouldEnforceStrictMixedContentChecking(true);

[alexmos, 2015/12/03 20:43:26]

It seems that Blink only ever flips the bit from false to true, but the
replication API is more generic and allows flipping it from true to false as
well.  Do we need that, or should the plumbing be more of the form
didEnforceStrictMixedContentChecking()?

[estark, 2015/12/04 00:19:31]

Done for the render frame -> browser API. The browser -> render frame proxy API
does need to be able to flip it from true to false on navigations, though.

[alexmos, 2015/12/04 23:41:11]

Acknowledged.

+}
+
 DEFINE_TRACE(Document)
 {
 #if ENABLE(OILPAN)
     visitor->trace(m_importsController);
     visitor->trace(m_docType);
     visitor->trace(m_implementation);
     visitor->trace(m_autofocusElement);
     visitor->trace(m_focusedElement);
     visitor->trace(m_hoverNode);
     visitor->trace(m_activeHoverElement);
@@ skipping 58 matching lines @@
 #ifndef NDEBUG
 using namespace blink;
 void showLiveDocumentInstances()
 {
     Document::WeakDocumentSet& set = Document::liveDocumentSet();
     fprintf(stderr, "There are %u documents currently alive:\n", set.size());
     for (Document* document : set)
         fprintf(stderr, "- Document %p URL: %s\n", document, document->url().string().utf8().data());
 }
 #endif