[webkit-dev] [webkit-changes] [56661] trunk

Robert Hogan lists at roberthogan.net
Mon Mar 29 13:44:43 PDT 2010


On Monday 29 March 2010 21:28:15 Adam Barth wrote:
> I think this change is wrong.  We should just pass the mime type along
> with the allowPlugins call and let the client decide what to do
> instead of teaching WebCore about some client-level concept.
> 
> Adam
> 

OK. I'll roll it out and make the rule specific to the Qt port unless 
someone shouts stop!

Thanks,
Robert

> On Mon, Mar 29, 2010 at 1:16 PM, Alexey Proskuryakov <ap at webkit.org> 
wrote:
> > What is an "application plugin" outside of Qt context? This change
> > will make all port authors scratch their heads trying to understand
> > what they want to return from isApplicationPluginMIMEType().
> > I think that this should be more obviously Qt-specific.
> > - WBR, Alexey Proskuryakov
> > On 27.03.2010, at 5:46, robert at webkit.org wrote:
> >
> > Revision56661Authorrobert at webkit.orgDate2010-03-27 05:46:35 -0700
> > (Sat, 27 Mar 2010)
> >
> > Log Message
> >
> > 2010-03-26  Robert Hogan  <robert at roberthogan.net>
> >
> >         Reviewed by Simon Hausmann.
> >
> >         Allow plugins implemented by the application, such as mimetype
> > 'x-qt-plugin',
> >          when pluginsEnabled is false.
> >
> >         The purpose of disabling plugins is to prevent the execution
> > of third-party code
> >         that may be untrustworthy. Qt plugins are implemented by the
> > client rather than
> >         loaded from an external source, so the client should have the
> > opportunity to
> >         consider them separately from other plugins.
> >
> >         Add a function MimeTypeRegistry::isApplicationPluginMIMEType()
> > that WebKit
> >         uses in conjunction with arePluginsEnabled() to determine if
> > it should attempt
> >         to load a plugin. If isApplicationPluginMIMEType() returns
> > true, WebKit will load
> >         the plugin even if arePluginsEnabled() is false.
> >
> >         Currently, only Qt has application-implemented plugins: these
> > use the mimetype
> >         'x-qt-plugin' and 'x-qt-styled-widget'. This patch permits Qt
> > clients'
> >         reimplementation of QWebPage::createPlugin() to decide whether
> > or not
> >         to create a Qt plugin, even when arePluginsEnabled is false.
> >
> >         For all platforms apart from Qt, isApplicationPluginMIMEType()
> > returns false.
> >
> >         https://bugs.webkit.org/show_bug.cgi?id=32196
> >
> >         Test: plugins/application-plugin-plugins-disabled.html
> >
> >         * loader/FrameLoader.cpp:
> >         (WebCore::FrameLoader::requestObject):
> >         * platform/MIMETypeRegistry.h:
> >         * platform/brew/MIMETypeRegistryBrew.cpp:
> >         (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> >         * platform/chromium/MIMETypeRegistryChromium.cpp:
> >         (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> >         * platform/gtk/MIMETypeRegistryGtk.cpp:
> >         (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> >         * platform/haiku/MIMETypeRegistryHaiku.cpp:
> >         (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> >         * platform/mac/MIMETypeRegistryMac.mm:
> >         (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> >         * platform/qt/MIMETypeRegistryQt.cpp:
> >         (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> >         * platform/win/MIMETypeRegistryWin.cpp:
> >         (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> >         * platform/wince/MIMETypeRegistryWince.cpp:
> >         (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> >         * platform/wx/MimeTypeRegistryWx.cpp:
> >         (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> >
> > 2010-03-26  Robert Hogan  <robert at roberthogan.net>
> >
> >         Reviewed by Simon Hausmann.
> >
> >         Allow plugins implemented by the application, such as mimetype
> > 'x-qt-plugin',
> >          when pluginsEnabled is false.
> >
> >         For all platforms apart from Qt, isApplicationPluginMIMEType()
> > returns false.
> >
> >         https://bugs.webkit.org/show_bug.cgi?id=32196
> >
> >         *
> > platform/qt/plugins/application-plugin-plugins-disabled-expected.txt:
> > Added. * plugins/application-plugin-plugins-disabled-expected.txt:
> > Added. * plugins/application-plugin-plugins-disabled.html: Added.
> >
> > 2010-03-26  Robert Hogan  <robert at roberthogan.net>
> >
> >         Reviewed by Simon Hausmann.
> >
> >         Allow plugins implemented by the application, such as mimetype
> > 'x-qt-plugin',
> >          when pluginsEnabled is false.
> >
> >         Add support for LayoutTestController.WebKitPluginsEnabled
> >
> >         https://bugs.webkit.org/show_bug.cgi?id=32196
> >
> >         * DumpRenderTree/gtk/DumpRenderTree.cpp:
> >         (resetDefaultsToConsistentValues):
> >         * DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:
> >         (copyWebSettingKey):
> >         * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
> >         (WebCore::WebPage::resetSettings):
> >         * DumpRenderTree/qt/LayoutTestControllerQt.cpp:
> >         (LayoutTestController::overridePreference):
> >
> > Modified Paths
> >
> > trunk/LayoutTests/ChangeLog
> > trunk/WebCore/ChangeLog
> > trunk/WebCore/loader/FrameLoader.cpp
> > trunk/WebCore/platform/MIMETypeRegistry.h
> > trunk/WebCore/platform/brew/MIMETypeRegistryBrew.cpp
> > trunk/WebCore/platform/chromium/MIMETypeRegistryChromium.cpp
> > trunk/WebCore/platform/gtk/MIMETypeRegistryGtk.cpp
> > trunk/WebCore/platform/haiku/MIMETypeRegistryHaiku.cpp
> > trunk/WebCore/platform/mac/MIMETypeRegistryMac.mm
> > trunk/WebCore/platform/qt/MIMETypeRegistryQt.cpp
> > trunk/WebCore/platform/win/MIMETypeRegistryWin.cpp
> > trunk/WebCore/platform/wince/MIMETypeRegistryWince.cpp
> > trunk/WebCore/platform/wx/MimeTypeRegistryWx.cpp
> > trunk/WebKitTools/ChangeLog
> > trunk/WebKitTools/DumpRenderTree/gtk/DumpRenderTree.cpp
> > trunk/WebKitTools/DumpRenderTree/gtk/LayoutTestControllerGtk.cpp
> > trunk/WebKitTools/DumpRenderTree/qt/DumpRenderTreeQt.cpp
> > trunk/WebKitTools/DumpRenderTree/qt/LayoutTestControllerQt.cpp
> >
> > Diff
> >
> > Modified: trunk/LayoutTests/ChangeLog (56660 => 56661)
> >
> > --- trunk/LayoutTests/ChangeLog	2010-03-27 12:36:03 UTC (rev 56660)
> > +++ trunk/LayoutTests/ChangeLog	2010-03-27 12:46:35 UTC (rev 56661)
> > @@ -1,3 +1,18 @@
> > +2010-03-26  Robert Hogan  <robert at roberthogan.net>
> > +
> > +        Reviewed by Simon Hausmann.
> > +
> > +        Allow plugins implemented by the application, such as
> > mimetype 'x-qt-plugin',
> > +         when pluginsEnabled is false.
> > +
> > +        For all platforms apart from Qt,
> > isApplicationPluginMIMEType() returns false.
> > +
> > +        https://bugs.webkit.org/show_bug.cgi?id=32196
> > +
> > +        *
> > platform/qt/plugins/application-plugin-plugins-disabled-expected.txt:
> > Added. +        *
> > plugins/application-plugin-plugins-disabled-expected.txt: Added. +    
> >    * plugins/application-plugin-plugins-disabled.html: Added. +
> >  2010-03-27  Yael Aharon  <yael.aharon at nokia.com>
> >
> >          Unreviewed.
> >
> > Modified: trunk/WebCore/ChangeLog (56660 => 56661)
> >
> > --- trunk/WebCore/ChangeLog	2010-03-27 12:36:03 UTC (rev 56660)
> > +++ trunk/WebCore/ChangeLog	2010-03-27 12:46:35 UTC (rev 56661)
> > @@ -1,3 +1,53 @@
> > +2010-03-26  Robert Hogan  <robert at roberthogan.net>
> > +
> > +        Reviewed by Simon Hausmann.
> > +
> > +        Allow plugins implemented by the application, such as
> > mimetype 'x-qt-plugin',
> > +         when pluginsEnabled is false.
> > +
> > +        The purpose of disabling plugins is to prevent the execution
> > of third-party code
> > +        that may be untrustworthy. Qt plugins are implemented by the
> > client rather than
> > +        loaded from an external source, so the client should have the
> > opportunity to
> > +        consider them separately from other plugins.
> > +
> > +        Add a function
> > MimeTypeRegistry::isApplicationPluginMIMEType() that WebKit
> > +        uses in conjunction with arePluginsEnabled() to determine if
> > it should attempt
> > +        to load a plugin. If isApplicationPluginMIMEType() returns
> > true, WebKit will load
> > +        the plugin even if arePluginsEnabled() is false.
> > +
> > +        Currently, only Qt has application-implemented plugins: these
> > use the mimetype
> > +        'x-qt-plugin' and 'x-qt-styled-widget'. This patch permits Qt
> > clients'
> > +        reimplementation of QWebPage::createPlugin() to decide
> > whether or not
> > +        to create a Qt plugin, even when arePluginsEnabled is false.
> > +
> > +        For all platforms apart from Qt,
> > isApplicationPluginMIMEType() returns false.
> > +
> > +        https://bugs.webkit.org/show_bug.cgi?id=32196
> > +
> > +        Test: plugins/application-plugin-plugins-disabled.html
> > +
> > +        * loader/FrameLoader.cpp:
> > +        (WebCore::FrameLoader::requestObject):
> > +        * platform/MIMETypeRegistry.h:
> > +        * platform/brew/MIMETypeRegistryBrew.cpp:
> > +        (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> > +        * platform/chromium/MIMETypeRegistryChromium.cpp:
> > +        (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> > +        * platform/gtk/MIMETypeRegistryGtk.cpp:
> > +        (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> > +        * platform/haiku/MIMETypeRegistryHaiku.cpp:
> > +        (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> > +        * platform/mac/MIMETypeRegistryMac.mm:
> > +        (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> > +        * platform/qt/MIMETypeRegistryQt.cpp:
> > +        (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> > +        * platform/win/MIMETypeRegistryWin.cpp:
> > +        (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> > +        * platform/wince/MIMETypeRegistryWince.cpp:
> > +        (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> > +        * platform/wx/MimeTypeRegistryWx.cpp:
> > +        (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
> > +
> >  2010-03-27  Dan Bernstein  <mitz at apple.com>
> >
> >          Reviewed by Mark Rowe.
> >
> > Modified: trunk/WebCore/loader/FrameLoader.cpp (56660 => 56661)
> >
> > --- trunk/WebCore/loader/FrameLoader.cpp	2010-03-27 12:36:03 UTC (rev
> > 56660) +++ trunk/WebCore/loader/FrameLoader.cpp	2010-03-27 12:46:35
> > UTC (rev 56661) @@ -1285,6 +1285,10 @@
> >      if (shouldUsePlugin(completedURL, mimeType,
> > renderer->hasFallbackContent(), useFallback)) {
> >          Settings* settings = m_frame->settings();
> >          if (!allowPlugins(AboutToInstantiatePlugin)
> > +             // Application plugins are plugins implemented by the
> > user agent, for example Qt plugins,
> > +             // as opposed to third-party code such as flash. The
> > user agent decides whether or not they are
> > +             // permitted, rather than WebKit.
> > +             &&
> > !MIMETypeRegistry::isApplicationPluginMIMEType(mimeType)
> >
> >              || (!settings->isJavaEnabled() &&
> >
> > MIMETypeRegistry::isJavaAppletMIMEType(mimeType)))
> >              return false;
> >          if (isDocumentSandboxed(SandboxPlugins))
> >
> > Modified: trunk/WebCore/platform/MIMETypeRegistry.h (56660 => 56661)
> >
> > --- trunk/WebCore/platform/MIMETypeRegistry.h	2010-03-27 12:36:03 UTC
> > (rev 56660)
> > +++ trunk/WebCore/platform/MIMETypeRegistry.h	2010-03-27 12:46:35 UTC
> > (rev 56661)
> > @@ -67,6 +67,10 @@
> >      // Check to see if a mime type is a valid Java applet mime type
> >      static bool isJavaAppletMIMEType(const String& mimeType);
> >
> > +    // Check to see if a mime type is a plugin implemented by the
> > +    // browser (e.g. a Qt Plugin).
> > +    static bool isApplicationPluginMIMEType(const String& mimeType);
> > +
> >      static HashSet<String>& getSupportedImageMIMETypes();
> >      static HashSet<String>& getSupportedImageResourceMIMETypes();
> >      static HashSet<String>& getSupportedImageMIMETypesForEncoding();
> >
> > Modified: trunk/WebCore/platform/brew/MIMETypeRegistryBrew.cpp (56660
> > => 56661)
> >
> > --- trunk/WebCore/platform/brew/MIMETypeRegistryBrew.cpp	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++ trunk/WebCore/platform/brew/MIMETypeRegistryBrew.cpp	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -76,5 +76,10 @@
> >      return "text/plain";
> >  }
> >
> > +bool MIMETypeRegistry::isApplicationPluginMIMEType(const String&)
> > +{
> > +    return false;
> > +}
> > +
> >  } // namespace WebCore
> >
> >
> > Modified: trunk/WebCore/platform/chromium/MIMETypeRegistryChromium.cpp
> > (56660 => 56661)
> >
> > ---
> > trunk/WebCore/platform/chromium/MIMETypeRegistryChromium.cpp	2010-03-2
> >7 12:36:03 UTC (rev 56660)
> > +++
> > trunk/WebCore/platform/chromium/MIMETypeRegistryChromium.cpp	2010-03-2
> >7 12:46:35 UTC (rev 56661)
> > @@ -130,6 +130,11 @@
> >      return String();
> >  }
> >
> > +bool MIMETypeRegistry::isApplicationPluginMIMEType(const String&)
> > +{
> > +    return false;
> > +}
> > +
> >  static HashSet<String>& dummyHashSet()
> >  {
> >      ASSERT_NOT_REACHED();
> >
> > Modified: trunk/WebCore/platform/gtk/MIMETypeRegistryGtk.cpp (56660 =>
> > 56661)
> >
> > --- trunk/WebCore/platform/gtk/MIMETypeRegistryGtk.cpp	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++ trunk/WebCore/platform/gtk/MIMETypeRegistryGtk.cpp	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -73,4 +73,9 @@
> >      return String();
> >  }
> >
> > +bool MIMETypeRegistry::isApplicationPluginMIMEType(const String&)
> > +{
> > +    return false;
> >  }
> > +
> > +}
> >
> > Modified: trunk/WebCore/platform/haiku/MIMETypeRegistryHaiku.cpp
> > (56660 => 56661)
> >
> > --- trunk/WebCore/platform/haiku/MIMETypeRegistryHaiku.cpp	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++ trunk/WebCore/platform/haiku/MIMETypeRegistryHaiku.cpp	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -71,5 +71,10 @@
> >      // unknown, let's just assume plain text
> >      return "text/plain";
> >  }
> > +
> > +bool MIMETypeRegistry::isApplicationPluginMIMEType(const String&)
> > +{
> > +    return false;
> > +}
> >  } // namespace WebCore
> >
> >
> > Modified: trunk/WebCore/platform/mac/MIMETypeRegistryMac.mm (56660 =>
> > 56661)
> >
> > --- trunk/WebCore/platform/mac/MIMETypeRegistryMac.mm	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++ trunk/WebCore/platform/mac/MIMETypeRegistryMac.mm	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -56,4 +56,9 @@
> >      return wkGetPreferredExtensionForMIMEType(type);
> >  }
> >
> > +bool MIMETypeRegistry::isApplicationPluginMIMEType(const String&)
> > +{
> > +    return false;
> >  }
> > +
> > +}
> >
> > Modified: trunk/WebCore/platform/qt/MIMETypeRegistryQt.cpp (56660 =>
> > 56661)
> >
> > --- trunk/WebCore/platform/qt/MIMETypeRegistryQt.cpp	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++ trunk/WebCore/platform/qt/MIMETypeRegistryQt.cpp	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -82,4 +82,10 @@
> >      return "application/octet-stream";
> >  }
> >
> > +bool MIMETypeRegistry::isApplicationPluginMIMEType(const String&
> > mimeType) +{
> > +    return mimeType.startsWith("application/x-qt-plugin", false)
> > +        || mimeType.startsWith("application/x-qt-styled-widget",
> > false); }
> > +
> > +}
> >
> > Modified: trunk/WebCore/platform/win/MIMETypeRegistryWin.cpp (56660 =>
> > 56661)
> >
> > --- trunk/WebCore/platform/win/MIMETypeRegistryWin.cpp	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++ trunk/WebCore/platform/win/MIMETypeRegistryWin.cpp	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -103,4 +103,9 @@
> >      return result;
> >  }
> >
> > +bool MIMETypeRegistry::isApplicationPluginMIMEType(const String&)
> > +{
> > +    return false;
> >  }
> > +
> > +}
> >
> > Modified: trunk/WebCore/platform/wince/MIMETypeRegistryWince.cpp
> > (56660 => 56661)
> >
> > --- trunk/WebCore/platform/wince/MIMETypeRegistryWince.cpp	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++ trunk/WebCore/platform/wince/MIMETypeRegistryWince.cpp	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -133,4 +133,9 @@
> >      return result.isEmpty() ? "unknown/unknown" : result;
> >  }
> >
> > +bool MIMETypeRegistry::isApplicationPluginMIMEType(const String&)
> > +{
> > +    return false;
> >  }
> > +
> > +}
> >
> > Modified: trunk/WebCore/platform/wx/MimeTypeRegistryWx.cpp (56660 =>
> > 56661)
> >
> > --- trunk/WebCore/platform/wx/MimeTypeRegistryWx.cpp	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++ trunk/WebCore/platform/wx/MimeTypeRegistryWx.cpp	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -70,4 +70,9 @@
> >      return "text/plain";
> >  }
> >
> > +bool MIMETypeRegistry::isApplicationPluginMIMEType(const String&)
> > +{
> > +    return false;
> >  }
> > +
> > +}
> >
> > Modified: trunk/WebKitTools/ChangeLog (56660 => 56661)
> >
> > --- trunk/WebKitTools/ChangeLog	2010-03-27 12:36:03 UTC (rev 56660)
> > +++ trunk/WebKitTools/ChangeLog	2010-03-27 12:46:35 UTC (rev 56661)
> > @@ -1,3 +1,23 @@
> > +2010-03-26  Robert Hogan  <robert at roberthogan.net>
> > +
> > +        Reviewed by Simon Hausmann.
> > +
> > +        Allow plugins implemented by the application, such as
> > mimetype 'x-qt-plugin',
> > +         when pluginsEnabled is false.
> > +
> > +        Add support for LayoutTestController.WebKitPluginsEnabled
> > +
> > +        https://bugs.webkit.org/show_bug.cgi?id=32196
> > +
> > +        * DumpRenderTree/gtk/DumpRenderTree.cpp:
> > +        (resetDefaultsToConsistentValues):
> > +        * DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:
> > +        (copyWebSettingKey):
> > +        * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
> > +        (WebCore::WebPage::resetSettings):
> > +        * DumpRenderTree/qt/LayoutTestControllerQt.cpp:
> > +        (LayoutTestController::overridePreference):
> > +
> >  2010-03-26  Adam Barth  <abarth at webkit.org>
> >
> >          Reviewed by Eric Seidel.
> >
> > Modified: trunk/WebKitTools/DumpRenderTree/gtk/DumpRenderTree.cpp
> > (56660 => 56661)
> >
> > --- trunk/WebKitTools/DumpRenderTree/gtk/DumpRenderTree.cpp	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++ trunk/WebKitTools/DumpRenderTree/gtk/DumpRenderTree.cpp	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -341,6 +341,7 @@
> >                   "enable-page-cache", FALSE,
> >                   "auto-resize-window", TRUE,
> >                   "enable-java-applet", FALSE,
> > +                 "enable-plugins", TRUE,
> >                   NULL);
> >
> >      webkit_web_frame_clear_main_frame_name(mainFrame);
> >
> > Modified:
> > trunk/WebKitTools/DumpRenderTree/gtk/LayoutTestControllerGtk.cpp
> > (56660 => 56661)
> >
> > ---
> > trunk/WebKitTools/DumpRenderTree/gtk/LayoutTestControllerGtk.cpp	2010-
> >03-27 12:36:03 UTC (rev 56660)
> > +++
> > trunk/WebKitTools/DumpRenderTree/gtk/LayoutTestControllerGtk.cpp	2010-
> >03-27 12:46:35 UTC (rev 56661)
> > @@ -66,12 +66,13 @@
> >
> >      if (!keyTable) {
> >          // If you add a pref here, make sure you reset the value in
> > -        //
> > DumpRenderTree::resetWebViewToConsistentStateBeforeTesting. +       
> > // DumpRenderTree::resetDefaultsToConsistentValues.
> >          keyTable = g_hash_table_new(g_str_hash, g_str_equal);
> >          g_hash_table_insert(keyTable,
> > g_strdup("WebKitJavaScriptEnabled"), g_strdup("enable-scripts"));
> >          g_hash_table_insert(keyTable,
> > g_strdup("WebKitDefaultFontSize"), g_strdup("default-font-size"));
> >          g_hash_table_insert(keyTable,
> > g_strdup("WebKitEnableCaretBrowsing"),
> > g_strdup("enable-caret-browsing")); g_hash_table_insert(keyTable,
> > g_strdup("WebKitUsesPageCachePreferenceKey"),
> > g_strdup("enable-page-cache"));
> > +        g_hash_table_insert(keyTable,
> > g_strdup("WebKitPluginsEnabled"), g_strdup("enable-plugins"));
> >      }
> >
> >      return g_strdup(static_cast<gchar*>(g_hash_table_lookup(keyTable,
> > preferenceKey)));
> >
> > Modified: trunk/WebKitTools/DumpRenderTree/qt/DumpRenderTreeQt.cpp
> > (56660 => 56661)
> >
> > ---
> > trunk/WebKitTools/DumpRenderTree/qt/DumpRenderTreeQt.cpp	2010-03-27
> > 12:36:03 UTC (rev 56660)
> > +++
> > trunk/WebKitTools/DumpRenderTree/qt/DumpRenderTreeQt.cpp	2010-03-27
> > 12:46:35 UTC (rev 56661)
> > @@ -188,6 +188,8 @@
> >     
> > settings()->resetAttribute(QWebSettings::LinksIncludedInFocusChain);
> >
> > settings()->resetAttribute(QWebSettings::OfflineWebApplicationCacheEna
> >bled);
> >
> > settings()->resetAttribute(QWebSettings::LocalContentCanAccessRemoteUr
> >ls); +    settings()->resetAttribute(QWebSettings::PluginsEnabled); +
> >      m_drt->layoutTestController()->setCaretBrowsingEnabled(false);
> >     
> > m_drt->layoutTestController()->setFrameSetFlatteningEnabled(false);
> >
> >
> > Modified:
> > trunk/WebKitTools/DumpRenderTree/qt/LayoutTestControllerQt.cpp (56660
> > => 56661)
> >
> > ---
> > trunk/WebKitTools/DumpRenderTree/qt/LayoutTestControllerQt.cpp	2010-03
> >-27 12:36:03 UTC (rev 56660)
> > +++
> > trunk/WebKitTools/DumpRenderTree/qt/LayoutTestControllerQt.cpp	2010-03
> >-27 12:46:35 UTC (rev 56661)
> > @@ -478,6 +478,8 @@
> >          QWebSettings::setMaximumPagesInCache(value.toInt());
> >      else if (name == "WebKitEnableCaretBrowsing")
> >          setCaretBrowsingEnabled(value.toBool());
> > +    else if (name == "WebKitPluginsEnabled")
> > +        settings->setAttribute(QWebSettings::PluginsEnabled,
> > value.toBool());
> >      else
> >          printf("ERROR: LayoutTestController::overridePreference()
> > does not support the '%s' preference\n",
> >              name.toLatin1().data());
> >
> > _______________________________________________
> > webkit-changes mailing list
> > webkit-changes at lists.webkit.org
> > http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
> >
> >
> >
> > _______________________________________________
> > webkit-dev mailing list
> > webkit-dev at lists.webkit.org
> > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
> 
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
> 


More information about the webkit-dev mailing list