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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 9 18:33:57 PDT 2010


Joseph Pecoraro <joepeck at webkit.org> changed:

           What    |Removed                     |Added
  Attachment #63935|review?                     |review+
               Flag|                            |

--- Comment #9 from Joseph Pecoraro <joepeck at webkit.org>  2010-08-09 18:33:57 PST ---
(From update of attachment 63935)
> Index: WebKit/chromium/src/ApplicationCacheHost.cpp
> +        if (page && page->inspectorController()->applicationCacheAgent() && page->mainFrame() == m_documentLoader->frame())

Patch looks good to me. Two points I wanted to comment on, which you may
have been aware of.

The "page->mainFrame()" check makes sense for now. I don't think we handle
this correctly in WebKit. I think once we support multiple frames / manifests
(see bug 41636) we will probably end up passing the frame into the calls to

Another point, it makes sense to ignore the progress event here, because
our UI doesn't do anything for progress events for now. But in the future
we might show some kind of progress, maybe a progress bar or #/# indicator.
I don't think there is a bug on that. I will file one now.

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