Changeset 70568 in webkit


Ignore:
Timestamp:
Oct 26, 2010 1:46:27 PM (13 years ago)
Author:
jberlin@webkit.org
Message:

Post any pending messages to the Injected Bundle in WebContext::processDidFinishLaunching
instead of in WebContext::ensureWebProcess.
https://bugs.webkit.org/show_bug.cgi?id=48367

Reviewed by Darin Adler.

Send pending injected bundle messages after the process finished launching instead of of
directly after creating it.

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::ensureWebProcess):
Move the logic to dispatch pending injected bundle messages from here ...
(WebKit::WebContext::processDidFinishLaunching):
... to here.
Also use a reference instead of a pointer when sending the pending messages.

Location:
trunk/WebKit2
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebKit2/ChangeLog

    r70564 r70568  
     12010-10-26  Jessie Berlin  <jberlin@apple.com>
     2
     3        Reviewed by Darin Adler.
     4
     5        Post any pending messages to the Injected Bundle in WebContext::processDidFinishLaunching
     6        instead of in WebContext::ensureWebProcess.
     7        https://bugs.webkit.org/show_bug.cgi?id=48367
     8
     9        Send pending injected bundle messages after the process finished launching instead of of
     10        directly after creating it.
     11
     12        * UIProcess/WebContext.cpp:
     13        (WebKit::WebContext::ensureWebProcess):
     14        Move the logic to dispatch pending injected bundle messages from here ...
     15        (WebKit::WebContext::processDidFinishLaunching):
     16        ... to here.
     17        Also use a reference instead of a pointer when sending the pending messages.
     18
    1192010-10-26  Anders Carlsson  <andersca@apple.com>
    220
  • trunk/WebKit2/UIProcess/WebContext.cpp

    r70359 r70568  
    179179
    180180    m_process->send(Messages::WebProcess::InitializeWebProcess(parameters), 0);
    181 
    182     for (size_t i = 0; i != m_pendingMessagesToPostToInjectedBundle.size(); ++i) {
    183         pair<String, RefPtr<APIObject> >* message = &m_pendingMessagesToPostToInjectedBundle[i];
    184         m_process->send(InjectedBundleMessage::PostMessage, 0, CoreIPC::In(message->first, WebContextUserMessageEncoder(message->second.get())));
    185     }
    186     m_pendingMessagesToPostToInjectedBundle.clear();
    187181}
    188182
     
    193187
    194188    m_visitedLinkProvider.populateVisitedLinksIfNeeded();
     189
     190    for (size_t i = 0; i != m_pendingMessagesToPostToInjectedBundle.size(); ++i) {
     191        pair<String, RefPtr<APIObject> >& message = m_pendingMessagesToPostToInjectedBundle[i];
     192        m_process->send(InjectedBundleMessage::PostMessage, 0, CoreIPC::In(message.first, WebContextUserMessageEncoder(message.second.get())));
     193    }
     194    m_pendingMessagesToPostToInjectedBundle.clear();
    195195}
    196196
Note: See TracChangeset for help on using the changeset viewer.