From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from resqmta-ch2-06v.sys.comcast.net (resqmta-ch2-06v.sys.comcast.net [IPv6:2001:558:fe21:29:69:252:207:38]) by hurricane.the-brannons.com (Postfix) with ESMTPS id 25CB977AF8 for ; Fri, 22 Jan 2016 04:40:15 -0800 (PST) Received: from resomta-ch2-14v.sys.comcast.net ([69.252.207.110]) by resqmta-ch2-06v.sys.comcast.net with comcast id 90hU1s0012PT3Qt010hY6B; Fri, 22 Jan 2016 12:41:32 +0000 Received: from eklhad ([IPv6:2601:405:4001:e487:21e:4fff:fec2:a0f1]) by resomta-ch2-14v.sys.comcast.net with comcast id 90hX1s00E2MDcd7010hXwb; Fri, 22 Jan 2016 12:41:31 +0000 To: edbrowse-dev@lists.the-brannons.com From: Karl Dahlke Reply-to: Karl Dahlke References: <20160121124908.GC1722@Kraftkrust> User-Agent: edbrowse/3.6.1+ Date: Fri, 22 Jan 2016 07:41:31 -0500 Message-ID: <20160022074131.eklhad@comcast.net> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=comcast.net; s=q20140121; t=1453466492; bh=MllmvEPaiOyINxFzpiqmq6AJLBQneLLZYq1W1g+uWEo=; h=Received:Received:To:From:Reply-to:Subject:Date:Message-ID: Mime-Version:Content-Type; b=Y8foov7yh1CAZU0mGB7P1pCoE/PD7aXwV6I15blTG9Yp9D9rpR4FT+0FaUv+LMQMS Oib5AKto/VH8hN0tiLflw2cGzbxlUJDrzhvUlkwSHPU1G3qYWlwnH6E87dXnDIBbhg qbK1/elWex9MulRiLixGdcT+FbLisi6p/lDxuXJNMtZxxpFRVkgTsGKsBh0+5Eghe9 lr3deTsE/QE6CTaSKQ294Tdaer0l9qSUck/H7Ioj0BG3AcC8xn9SfccMoBR/FS/TuH u6DcBCXARpt5AbVGJk/OXqX4yhX0MCk9idSebjzHiPcaOB8SAO0566mZ6uD4z/11om 3vw/830NP7jUw== Subject: [Edbrowse-dev] JS interpreter crashes on Google books X-BeenThere: edbrowse-dev@lists.the-brannons.com X-Mailman-Version: 2.1.20 Precedence: list List-Id: Edbrowse Development List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 22 Jan 2016 12:40:15 -0000 Well gdb cuts to the heart of it, as it often does. The problem is in the new xhr system when cw (current window) is null. And why didn't we see this in our jsrt test? Because that test runs innerHTML first, which sets cw. This page has no innerHTML, cw was never set, then xhr blows up. So I now set cw at the start of js_main(). Not sure the result is much better though. js script doesn't run perfectly, which reports an error to google via xhr, and too many errors like this can cause google to block your site, for a while. google is so complicated, it's not clear edbrowse will ever truly be able to handle it. This page does however have {toggle screen reader mode}. Not sure what that does. Karl Dahlke