https://bugs.webkit.org/show_bug.cgi?id=67303
Also removes some superfluous guards and methods added for Android.
Reviewed by Darin Adler.
No new tests, removing dead code only.
* WebCorePrefix.h:
Chromium uses this file only on Mac, so these Android guards are no
longer required.
* bindings/v8/NPV8Object.cpp:
* bindings/v8/NPV8Object.h:
Including npruntime.h for non-Chromium ports was added in
http://trac.webkit.org/changeset/53634 for Android. However, this
file is included in WebCore.pro, so leaving the guard in place.
* bindings/v8/ScriptCachedFrameData.h:
This code was added for Android in
http://trac.webkit.org/changeset/56716 but is now also used by QT. The
comment should have been removed in
http://trac.webkit.org/changeset/93573.
* dom/Document.cpp:
(WebCore::Document::Document):
(WebCore::Document::isLayoutTimerActive):
(WebCore::Document::minimumLayoutDelay):
* dom/Document.h:
This method was added for Android in
http://trac.webkit.org/changeset/52919 and modified in
http://trac.webkit.org/changeset/79775. It is no longer needed.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@94290
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2011-08-31 Steve Block <steveblock@google.com>
+
+ Clean up all mentions of Android in comments
+ https://bugs.webkit.org/show_bug.cgi?id=67303
+
+ Also removes some superfluous guards and methods added for Android.
+
+ Reviewed by Darin Adler.
+
+ No new tests, removing dead code only.
+
+ * WebCorePrefix.h:
+ Chromium uses this file only on Mac, so these Android guards are no
+ longer required.
+ * bindings/v8/NPV8Object.cpp:
+ * bindings/v8/NPV8Object.h:
+ Including npruntime.h for non-Chromium ports was added in
+ http://trac.webkit.org/changeset/53634 for Android. However, this
+ file is included in WebCore.pro, so leaving the guard in place.
+ * bindings/v8/ScriptCachedFrameData.h:
+ This code was added for Android in
+ http://trac.webkit.org/changeset/56716 but is now also used by QT. The
+ comment should have been removed in
+ http://trac.webkit.org/changeset/93573.
+ * dom/Document.cpp:
+ (WebCore::Document::Document):
+ (WebCore::Document::isLayoutTimerActive):
+ (WebCore::Document::minimumLayoutDelay):
+ * dom/Document.h:
+ This method was added for Android in
+ http://trac.webkit.org/changeset/52919 and modified in
+ http://trac.webkit.org/changeset/79775. It is no longer needed.
+
2011-09-01 Kristóf Kosztyó <kkristof@inf.u-szeged.hu>
[Qt] Build dependency problems
#include <time.h>
-#if !defined(BUILDING_WX__) && !defined(ANDROID) && !defined(BUILDING_BREWMP__)
+#if !defined(BUILDING_WX__) && !defined(BUILDING_BREWMP__)
#include <CoreFoundation/CoreFoundation.h>
#ifdef WTF_PLATFORM_WIN_CAIRO
#include <ConditionalMacros.h>
#endif
#endif
-#endif // !defined(BUILDING_WX__) && !defined(ANDROID)
+#endif // !defined(BUILDING_WX__) && !defined(BUILDING_BREWMP__)
#ifdef __OBJC__
#import <Cocoa/Cocoa.h>
#include "npruntime_impl.h"
#include "npruntime_priv.h"
-#if PLATFORM(CHROMIUM)
-#include <bindings/npruntime.h>
-#else
-#include "npruntime.h"
-#endif
-
#include <stdio.h>
#include <wtf/StringExtras.h>
// moved into bridge/npruntime.h, guarded by an #if PlATFORM(CHROMIUM).
#include <bindings/npruntime.h>
#else
-#include "npruntime.h" // Use WebCore version for Android and other ports.
+#include "npruntime.h" // Use WebCore version for other ports.
#endif
#include <v8.h>
#elif PLATFORM(QT)
// FIXME: the right guard should be ENABLE(PAGE_CACHE). Replace with the right guard, once
// https://bugs.webkit.org/show_bug.cgi?id=35061 is fixed.
-//
-// On Android we do use WebKit's page cache. For now we don't support isolated worlds
-// so our implementation does not take them into account.
#include "OwnHandle.h"
#include <v8.h>
, m_loadEventFinished(false)
, m_startTime(currentTime())
, m_overMinimumLayoutThreshold(false)
- , m_extraLayoutDelay(0)
, m_scriptRunner(ScriptRunner::create(this))
, m_xmlVersion("1.0")
, m_xmlStandalone(false)
bool Document::isLayoutTimerActive()
{
- if (!view() || !view()->layoutPending())
- return false;
- bool isPendingLayoutImmediate = minimumLayoutDelay() == m_extraLayoutDelay;
- return isPendingLayoutImmediate;
+ return view() && view()->layoutPending() && !minimumLayoutDelay();
}
int Document::minimumLayoutDelay()
{
if (m_overMinimumLayoutThreshold)
- return m_extraLayoutDelay;
+ return 0;
int elapsed = elapsedTime();
m_overMinimumLayoutThreshold = elapsed > cLayoutScheduleThreshold;
// We'll want to schedule the timer to fire at the minimum layout threshold.
- return max(0, cLayoutScheduleThreshold - elapsed) + m_extraLayoutDelay;
+ return max(0, cLayoutScheduleThreshold - elapsed);
}
int Document::elapsedTime() const
bool parsing() const { return m_bParsing; }
int minimumLayoutDelay();
- // This method is used by Android.
- void setExtraLayoutDelay(int delay) { m_extraLayoutDelay = delay; }
-
bool shouldScheduleLayout();
bool isLayoutTimerActive();
int elapsedTime() const;
RefPtr<SerializedScriptValue> m_pendingStateObject;
double m_startTime;
bool m_overMinimumLayoutThreshold;
- // This is used to increase the minimum delay between re-layouts. It is set
- // using setExtraLayoutDelay to modify the minimum delay used at different
- // points during the lifetime of the Document.
- int m_extraLayoutDelay;
OwnPtr<ScriptRunner> m_scriptRunner;