Opened on 05/08/2014 at 09:16:37 PM

Closed on 05/14/2014 at 03:01:02 PM

Last modified on 01/10/2015 at 11:33:05 AM

#450 closed defect (fixed)

chrome: print a spreadsheet crashes the page

Reported by: mapx Assignee: sebastian
Priority: P1 Milestone: Adblock-Plus-1.8.3-for-Chrome-Opera-Safari
Module: Platform Keywords:
Cc: trev Blocked By:
Blocking: Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

Description (last modified by mapx)


chrome 34, 36
ABP 1.8

How to reproduce

  1. create a spreadsheet on google drive
  2. print the spreadsheet
  3. easylist ON ==> page goes in crash in chrome 34, or operation does not end chrome 36 (never ending "loading preview")

reported here:

easylist OFF ==> the operation concludes normally

Attachments (2) (562 bytes) - added by sebastian on 05/09/2014 at 11:14:55 AM. (819 bytes) - added by trev on 05/09/2014 at 12:43:11 PM.
Extension to reproduce the issue

Download all attachments as: .zip

Change History (17)

comment:1 Changed on 05/08/2014 at 09:17:19 PM by mapx

  • Description modified (diff)

comment:2 Changed on 05/09/2014 at 06:42:54 AM by mapx

  • Component changed from Unknown to Platform

comment:3 Changed on 05/09/2014 at 07:45:49 AM by trev

Interesting, I can in fact reproduce that issue with a new (empty) spreadsheet - but not with an existing larger spreadsheet. Adding @@||^$elemhide as a filter to disable element hiding on that site "fixes" the issue, Chrome no longer crashes. So I suspect that the issue a bug in Chrome's shadow DOM implementation.

comment:4 Changed on 05/09/2014 at 08:06:44 AM by trev

I tried to reproduce the issue without Adblock Plus by creating the shadow root from the web tools console but without success so far. Sebastian, maybe you will have more luck here.

Changed on 05/09/2014 at 11:14:55 AM by sebastian

comment:5 Changed on 05/09/2014 at 11:16:13 AM by sebastian

I can not reproduce it with Chrome 34. However I have uploaded a dummy extension that injects following code:


Can you please check whether you get the same behavior without Adblock Plus, but that extension installed?

Changed on 05/09/2014 at 12:43:11 PM by trev

Extension to reproduce the issue

comment:6 Changed on 05/09/2014 at 12:46:33 PM by trev

No, your extension didn't work for me - I attached a modified version, the issue can be partially reproduced with this one. When I try to print the spreadsheet the page starts loading something and hangs, if I abort the load it crashes. The important thing here is a message exchange with the background page, the issue only occurs if the shadow root is created in the response callback. For some reason I was only able to reproduce the immediate crash with Adblock Plus so far, the specifics of this issue look like memory corruption in Chrome to me.

comment:7 Changed on 05/09/2014 at 02:17:07 PM by sebastian

I reproduced it and filed a Chrome bug.

Do you think that case is worth adding a check, not using Shadow DOM on, as workaround?

Last edited on 05/09/2014 at 02:31:09 PM by sebastian

comment:8 Changed on 05/09/2014 at 04:05:12 PM by trev

As long as we don't understand the issue - no, that's not really a solution. We have no idea what causes this and whether other domains are affected.

comment:9 Changed on 05/14/2014 at 02:08:50 PM by sebastian

Also see #482

comment:10 Changed on 05/14/2014 at 02:28:28 PM by sebastian

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

Creating the Shadow DOM immediately instead inside the response callback fixes the issue. The only downside is that we have to create a shadow root even if Adblock Plus is disabled on the current page.

comment:11 Changed on 05/14/2014 at 03:00:41 PM by sebastian

  • Cc sebastian removed
  • Owner set to sebastian

comment:12 Changed on 05/14/2014 at 03:01:02 PM by sebastian

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:13 Changed on 05/15/2014 at 09:09:22 AM by sebastian

Also see #453

comment:14 Changed on 05/16/2014 at 01:44:13 PM by sebastian

  • Ready set

comment:15 Changed on 01/10/2015 at 11:33:05 AM by sebastian

  • Platform set to Unknown

Also see #452

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from sebastian.
Note: See TracTickets for help on using tickets.