Bug 102651 - [EFL] Fix assertion on processing ipc messages about network info
Summary: [EFL] Fix assertion on processing ipc messages about network info
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 131841
Blocks: 100310
  Show dependency treegraph
 
Reported: 2012-11-18 23:31 PST by Byungwoo Lee
Modified: 2014-04-18 03:56 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Byungwoo Lee 2012-11-18 23:31:50 PST
Need to fix assertion on processing ipc messages about network info.
This is the sub issue of Bug 100310.
Comment 1 Chris Dumez 2012-11-24 09:19:26 PST
Any update on this? Are you planning to write the patch or can I assign this bug to me?
Comment 2 Byungwoo Lee 2012-11-24 16:13:16 PST
Actually, I can't decide the solution for this.
In current implementation,

StartUpdating and StopUpdating messages of WebNetworkInfoManagerProxy are unified by WebNetworkInfoManager. (All WebPages share the IPC message, so WebProcess sent the IPC message without page destination ID)

But GetBandwidth and IsMetered is handled per WebPage. (WebProcess sends the IPC message with page destination ID, so the IPC messages need to be handled per WebPage)

To apply r132386,
1. IPC Messages of WebNetworkInfoManagerProxy need to be sorted.
   (like geolocation IPC, WebPages share the StartUpdating/StopUpdating, and WebPageProxy have GetBandwidth and IsMetered message)

2. Change for WebProcess to send all the network info IPC messages with page destination ID, and all the messages will be handled per WebPage.

Personally, I prefer the first solution.

How about your opinion? Is there any other solution?