Mobster fails sometimes when running against a desktop chrome about mobster HOT 8 CLOSED

We should get a static object first and then loop over it, instead of looping over the dict.

from mobster.

I just tested and this code change most likely fixes the issue:

diff --git a/src/linkedin/mobster/webkitcommunicator.py b/src/linkedin/mobster/webkitcommunicator.py
index b3abc40..543eb55 100644
--- a/src/linkedin/mobster/webkitcommunicator.py
+++ b/src/linkedin/mobster/webkitcommunicator.py
@@ -58,7 +58,8 @@ class RemoteWebKitCommunicator(WebSocketClient):
       id = response['id']
       self._response_callbacks[id](response)
     elif 'method' in response:
-      for callback in self._domain_callbacks[response['method'].split('.')[0]].itervalues():
+      callbacks = self._domain_callbacks[response['method'].split('.')[0]].values()
+      for callback in callbacks:
         callback(response)
     else:
       log.warn('Unrecognized message: {0}'.format(pformat(response)))

from mobster.

Is there any correlation between the webpage currently being navigated to and when this error occurs?

from mobster.

Haven't tested for other pages yet, only linkedin.

from mobster.

Oh I meant does the error normally occur on one particular page in the flow? I can see how this problem could occur, though I'm surprised it actually does.

If that code fixes the problem, please open a pull request (and feel free to merge it yourself).

from mobster.

Look at the other issue I opened (#19), that may solve this. Lets hold off until we have resolved that issue.

from mobster.

Confirming that fixing #19 does not fix this issue.

from mobster.

Ritesh can you please close this issue after you test it on your desktop

thanks
Hari

from mobster.