Bug 91540 - [BlackBerry] Remove unnecessary clearWindow() calls and the method itself
Summary: [BlackBerry] Remove unnecessary clearWindow() calls and the method itself
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jakob Petsovits
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-17 13:52 PDT by Jakob Petsovits
Modified: 2012-07-17 14:48 PDT (History)
5 users (show)

See Also:


Attachments
Patch (4.74 KB, patch)
2012-07-17 13:55 PDT, Jakob Petsovits
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jakob Petsovits 2012-07-17 13:52:09 PDT
While starting to fix the underlying issues of internal PR 174365 (assertion through clearWindow() call when we don't have any target buffer), I realized that this function call is actually entirely unnecessary. That doesn't obviate the need for a proper fix, but why not get rid of it when we have the chance. Patch below, more detailed explanation in the commit message.
Comment 1 Jakob Petsovits 2012-07-17 13:55:03 PDT
Created attachment 152820 [details]
Patch
Comment 2 WebKit Review Bot 2012-07-17 14:48:04 PDT
Comment on attachment 152820 [details]
Patch

Clearing flags on attachment: 152820

Committed r122876: <http://trac.webkit.org/changeset/122876>
Comment 3 WebKit Review Bot 2012-07-17 14:48:08 PDT
All reviewed patches have been landed.  Closing bug.