[Webkit-unassigned] [Bug 28359] Cleanup: Undo some accidental changes around DOMApplicationCache

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=28359


Alexey Proskuryakov <ap at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #39035|review?                     |review+
               Flag|                            |




--- Comment #6 from Alexey Proskuryakov <ap at webkit.org>  2009-09-04 13:42:31 PDT ---
(From update of attachment 39035)
r=me

For the record, the rationale behind this behavior is:

- it's slightly more consistent for a frameless window to be able to construct
a DOMApplicaitonCache, because window.applicationCache would be non-null if
constructed before detaching;
- it's unlikely that any browser will have ApplicationCache compiled in, but
disabled; run-time disabling is needed for clients like Dashboard. So, knowing
that it's non-null can simplify code a little, and will do no harm.

Neither is particularly important, just minor cleanup.

+        
layoutTestController.overridePreference("WebKitOfflineWebApplicationCacheEnabled",
false);

Does this really work on all platforms? You may need to add this new test to
Skipped list for some (or maybe the platforms that don't have it already skip
all tests http/tests/in appcache?)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.