[Webkit-unassigned] [Bug 43531] Web Inspector: Support appcache status change for Chrome

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Aug 7 03:10:58 PDT 2010


Pavel Feldman <pfeldman at chromium.org> changed:

           What    |Removed                     |Added
  Attachment #63647|review?                     |review-
               Flag|                            |

--- Comment #7 from Pavel Feldman <pfeldman at chromium.org>  2010-08-07 03:10:58 PST ---
(From update of attachment 63647)
 +          if (page && page->inspectorController()->inspectorFrontend() && page->mainFrame() == m_documentLoader->frame())
You should not make ApplicationCacheHost too much involved into the InspectorController's business. inspectorFrontend() should not be used as indication of the frontend availability + it will actually go away this week. You should notify inspector controller on any change and let it decide when to trigger the action.

 +              page->inspectorController()->applicationCacheAgent()->updateApplicationCacheStatus(status());
So you will simply check applicationCacheAgent for being 0, and if not, notify it.

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

More information about the webkit-unassigned mailing list