Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

chrome.tabs.onUpdated.addListener is getting complete status twice or more for each page

Here is my Script, which uses onUpdated.addListener to check each url:

chrome.tabs.onUpdated.addListener(function (tabId, changeInfo, tab) {          
 alert(changeInfo.status);
 if (changeInfo.status == 'complete') {
 if (tab.url.indexOf("in.yahoo.mail") !== -1) {
 alert(tab.url);
 chrome.tabs.update(tabId, { url: "https://accounts.google.com/ServiceLogin" });
  //injectToTab(tab);
  }
 }
}); 

And here is the Manifest code:

 {
  "name": "LeoPlugin For Test",
  "version": "1.6",
  "manifest_version": 2,
  "content_security_policy": "script-src 'self'; object-src 'self'",
  "description": "Extension to Automate.",
  "background": {
    "scripts": ["js/eventPage.js"],
    "persistent": false
  },
  "content_scripts": [
    {
      "matches": ["http://*/*"],
      "js": ["js/eventPage.js"]
    }
  ],
  "icons":{"48":"images/bob48.png", "128":"images/bob128.png"}, //Define any icon sizes and the files that you want to use with them. 48/128 etc.
  "browser_action": {
    "default_icon": "images/bob.png",       // What icon do you want to display on the chrome toolbar
    "default_popup": "testwatch.html"       // The page to popup when button clicked.
  },
  "permissions": [
    "tabs", "http://*/*","https://*/*"             // Cross Site Access Requests
  ]

}

Anything missing in the above code that can prevent getting 'complete' status more than once. And also here is my HTML code:

<html>
  <head>
    <title>Leo Chrome Extension</title>

    <link href="css/main.css" rel="stylesheet" type="text/css" />
    <link href="css/fonts.css" rel="stylesheet" type="text/css" />
    <script type="text/javascript" src="jquery/jquery-1.4.2.min.js"></script>
    <script type="text/javascript" src="jquery/jquery-ui-1.8.2.custom.min.js"></script>
    <!--<script type="text/javascript" src="js/Pluginjshelper.js"></script>-->
</head>
<body class="inner_bg">
        <div class="grey-panel">
            <div>
            <div class="user-pic">
                <img src="Images/leo-logo.png" width="70" height="50" />
            </div>
            <h3>
                Leo Extension</h3><input name="ok" type="button" class="btn" id = "button123"  value="OK"/> <!--onclick = "getCPRepGraph()" -->
                <input name="chkEnable" id="chkPluginEnable" type="checkbox" value="" class="check"  /><span class="lbl">Enable</span>
            </div>
            <div class="clear">
            </div>
            <div>
                <h4>
                    <span class="blue">Plugin </span>Credentials</h4>
                <ul class="addpage-form2">
                    <li>
                        <label class="addpage-label">
                            User Name :</label><input name="txtUserName" id="txtUserName" type="text" class="addpage-input2" />
                    </li>
                    <li>
                        <label class="addpage-label">
                            Password :</label><input name="txtPassword" id="txtPassword" type="password" class="addpage-input2" />
                    </li>
                </ul>
                <div class="clear">
                </div>
            </div>
            <!--BASIC INFO panel End-->
            <div class="clear">
            </div>
        </div>
</body>
</html>

Thanks in advance for your time.

like image 941
Vineel Gogineni Avatar asked Nov 30 '12 08:11

Vineel Gogineni


1 Answers

This was a known bug in Chrome Issue 162543 which was marked as fixed on 2012-12-05.

Workaround: (not needed anymore)

Make Event Page to Background Page by changing code from

"background": {
    "scripts": ["js/eventPage.js"],
    "persistent": false
  },

to

"background": {
    "scripts": ["js/eventPage.js"],
    "persistent": true
  },

and moving ahead by using background pages for development till the fix is delivered.

Other Points:

After a look at your manifest.json why does

"background": {
    "scripts": ["js/eventPage.js"],
    "persistent": false
  },
  "content_scripts": [
    {
      "matches": ["http://*/*"],
      "js": ["js/eventPage.js"]
    }
  ],

js/eventPage.js is made content script as well as background\event script; Having code for chrome.tabs.onUpdated.addListener() will not work in content scripts at all, please eliminate this code

Let me know if you need more information.

like image 68
Sudarshan Avatar answered Nov 03 '22 06:11

Sudarshan