Bug 154068 - Loading big websites causes the browser to crash
Summary: Loading big websites causes the browser to crash
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: Other
Hardware: All All
: P2 Major
Assignee: Andreas Kling
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2016-02-10 03:15 PST by paladox
Modified: 2016-03-01 11:41 PST (History)
8 users (show)

See Also:


Attachments
Patch (1.89 KB, patch)
2016-02-18 04:06 PST, Andreas Kling
no flags Details | Formatted Diff | Diff
Patch (1.89 KB, patch)
2016-02-18 04:14 PST, Andreas Kling
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description paladox 2016-02-10 03:15:29 PST
Hi loading large websites causes the browser to crash.

To reproduce please follow these steps

Use iOS 9.2 with an iPhone or iPad.

* go to https://gerrit.wikimedia.org/r/#/c/269651/ click on the layout.yaml file. When it's loading the browser will crash.

* since Google chrome is a fork of WebKit I tested Google chrome on Windows and manages to load the website without crashes and loads pretty fast. I presume the fix is somewhere in Google chrome. We should backported it here since reviewing changes is important even big files with little changes.
Comment 1 paladox 2016-02-10 03:17:21 PST
I've filed this bug here https://phabricator.wikimedia.org/T126442 for that file. But we Should fix the bug that causes loading large files through web browser to crash
Comment 2 Radar WebKit Bug Importer 2016-02-12 09:16:12 PST
<rdar://problem/24629358>
Comment 3 Simon Fraser (smfr) 2016-02-17 17:24:23 PST
What does "click on the layout.yaml file" mean? https://gerrit.wikimedia.org/r/#/c/269651/1/zuul/layout.yaml is a small document (a diff).
Comment 4 paladox 2016-02-17 17:26:38 PST
(In reply to comment #3)
> What does "click on the layout.yaml file" mean?
> https://gerrit.wikimedia.org/r/#/c/269651/1/zuul/layout.yaml is a small
> document (a diff).
It means to click on the file that I have linked meaning when you click on the link, click on the file to view the diff. The file will not load in WebKit it just crashed whereas it loads in chrome.
Comment 5 Simon Fraser (smfr) 2016-02-17 17:55:04 PST
Can you paste the actual url for the page that crashes on iOS but loads in Chrome?
Comment 6 paladox 2016-02-17 18:18:03 PST
(In reply to comment #5)
> Can you paste the actual url for the page that crashes on iOS but loads in
> Chrome?

Ok heres the link https://gerrit.wikimedia.org/r/#/c/270938/4/zuul/layout.yaml It works in chrome but crashes on ios probably it happends to webkit.
Comment 7 Simon Fraser (smfr) 2016-02-17 18:31:16 PST
Ah, that's the one I tried. It does indeed run out of memory on older devices.
Comment 8 paladox 2016-02-17 18:35:18 PST
(In reply to comment #7)
> Ah, that's the one I tried. It does indeed run out of memory on older
> devices.

Ok but it also happends on newer devices such as an ipad pro and a iPhone 6. Ipad pro is 128gb and iPhone 6 plus a 64gb.

Google chrome works on windows. I doint think it works on ios since it uses the same engine and safari.
Comment 9 Andreas Kling 2016-02-18 04:06:28 PST
Created attachment 271650 [details]
Patch
Comment 10 WebKit Commit Bot 2016-02-18 04:08:25 PST
Attachment 271650 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 11 Andreas Kling 2016-02-18 04:14:35 PST
Created attachment 271652 [details]
Patch
Comment 12 paladox 2016-02-18 04:19:53 PST
(In reply to comment #11)
> Created attachment 271652 [details]
> Patch

Thanks.
Comment 13 Simon Fraser (smfr) 2016-02-18 08:42:07 PST
Very nice.
Comment 14 WebKit Commit Bot 2016-02-18 09:25:40 PST
Comment on attachment 271652 [details]
Patch

Clearing flags on attachment: 271652

Committed r196761: <http://trac.webkit.org/changeset/196761>
Comment 15 WebKit Commit Bot 2016-02-18 09:25:44 PST
All reviewed patches have been landed.  Closing bug.
Comment 16 paladox 2016-02-23 10:34:52 PST
Hi do you know when this will be added To an iOS build so I can test to see if the crashed stop on the iPhone please.
Comment 17 paladox 2016-03-01 10:57:25 PST
If the fix is included in iOS 9.3 beta 5 then it isent working still crashing. If it is still to be included when will that be please.
Comment 18 Saam Barati 2016-03-01 11:41:41 PST
(In reply to comment #17)
> If the fix is included in iOS 9.3 beta 5 then it isent working still
> crashing. If it is still to be included when will that be please.

Apple does not comment on the timing or content of future releases.