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

Mon Aug 9 18:33:57 PDT 2010

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

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
applicationCacheAgent.

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.