Attachment 164748 Details for Bug 96521 – Patch

[patch] Patch

bug-96521-20120919122235.patch (text/plain), 3.49 KB, created by Mark Pilgrim (Google) on 2012-09-19 09:23:10 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Pilgrim (Google)

Created: 2012-09-19 09:23:10 PDT

Size: 3.49 KB

patch

obsolete

>Index: Source/WebCore/ChangeLog
>===================================================================
>--- Source/WebCore/ChangeLog	(revision 129009)
>+++ Source/WebCore/ChangeLog	(working copy)
>@@ -1,3 +1,19 @@
>+2012-09-19  Mark Pilgrim  <pilgrim@chromium.org>
>+
>+        [Chromium] Remove unused popupsAllowed function from PlatformSupport
>+        https://bugs.webkit.org/show_bug.cgi?id=96521
>+
>+        Reviewed by Eric Seidel.
>+
>+        Part of a refactoring series. See tracking bug 82948.
>+
>+        * bindings/v8/NPV8Object.cpp:
>+        (WebCore::v8ObjectToNPObject):
>+        (_NPN_Evaluate):
>+        (_NPN_GetProperty):
>+        * platform/chromium/PlatformSupport.h:
>+        (PlatformSupport):
>+
> 2012-09-19  Raul Hudea  <rhudea@adobe.com>
> 
>         [CSSRegions][CSSOM] Make sure all Regions APIs are not visible if CSS_REGIONS is not defined
>Index: Source/WebCore/bindings/v8/NPV8Object.cpp
>===================================================================
>--- Source/WebCore/bindings/v8/NPV8Object.cpp	(revision 129009)
>+++ Source/WebCore/bindings/v8/NPV8Object.cpp	(working copy)
>@@ -28,7 +28,6 @@
> 
> #include "NPV8Object.h"
> 
>-#include "PlatformSupport.h"
> #include "DOMWindow.h"
> #include "Frame.h"
> #include "NPObjectWrapper.h"
>@@ -296,7 +295,8 @@ bool _NPN_InvokeDefault(NPP npp, NPObjec
> 
> bool _NPN_Evaluate(NPP npp, NPObject* npObject, NPString* npScript, NPVariant* result)
> {
>-    bool popupsAllowed = PlatformSupport::popupsAllowed(npp);
>+    // FIXME: Give the embedder a way to control this.
>+    bool popupsAllowed = false;
>     return _NPN_EvaluateHelper(npp, popupsAllowed, npObject, npScript, result);
> }
> 
>Index: Source/WebCore/platform/chromium/PlatformSupport.h
>===================================================================
>--- Source/WebCore/platform/chromium/PlatformSupport.h	(revision 129009)
>+++ Source/WebCore/platform/chromium/PlatformSupport.h	(working copy)
>@@ -119,7 +119,6 @@ public:
>     // Plugin -------------------------------------------------------------
>     static bool plugins(bool refresh, Vector<PluginInfo>*);
>     static NPObject* pluginScriptableObject(Widget*);
>-    static bool popupsAllowed(NPP);
> 
>     // Screen -------------------------------------------------------------
>     static int screenHorizontalDPI(Widget*);
>Index: Source/WebKit/chromium/ChangeLog
>===================================================================
>--- Source/WebKit/chromium/ChangeLog	(revision 129009)
>+++ Source/WebKit/chromium/ChangeLog	(working copy)
>@@ -1,3 +1,15 @@
>+2012-09-19  Mark Pilgrim  <pilgrim@chromium.org>
>+
>+        [Chromium] Remove unused popupsAllowed function from PlatformSupport
>+        https://bugs.webkit.org/show_bug.cgi?id=96521
>+
>+        Reviewed by Eric Seidel.
>+
>+        Part of a refactoring series. See tracking bug 82948.
>+
>+        * src/PlatformSupport.cpp:
>+        (WebCore):
>+
> 2012-09-19  Kent Tamura  <tkent@chromium.org>
> 
>         [Chromium] Fix crash in WebFrameImpl::loadHistoryItem
>Index: Source/WebKit/chromium/src/PlatformSupport.cpp
>===================================================================
>--- Source/WebKit/chromium/src/PlatformSupport.cpp	(revision 129009)
>+++ Source/WebKit/chromium/src/PlatformSupport.cpp	(working copy)
>@@ -566,12 +566,6 @@ IntRect PlatformSupport::screenAvailable
>     return client->screenInfo().availableRect;
> }
> 
>-bool PlatformSupport::popupsAllowed(NPP npp)
>-{
>-    // FIXME: Give the embedder a way to control this.
>-    return false;
>-}
>-
> #if ENABLE(WORKERS)
> WorkerContextProxy* WorkerContextProxy::create(Worker* worker)
> {

Actions: View | Formatted Diff | Diff

Attachments on bug 96521: 163635 | 164748