Skip to content

Improve range creation#1133

Merged
eocanha merged 1 commit intowpe-2.38from
pgorszkowski/wpe-2.38/improve_range_creation
Aug 4, 2023
Merged

Improve range creation#1133
eocanha merged 1 commit intowpe-2.38from
pgorszkowski/wpe-2.38/improve_range_creation

Conversation

@pgorszkowski-igalia
Copy link
Copy Markdown

https://bugs.webkit.org/show_bug.cgi?id=258866

Reviewed by Xabier Rodriguez-Calvar.

Before this change, finding the place where new range should be added/merged/created was done with linear approach.
This change provides logarithmic approach to find the place for new range.

  • Source/WebCore/platform/graphics/PlatformTimeRanges.cpp: (WebCore::PlatformTimeRanges::add):
    (WebCore::PlatformTimeRanges::findLastRangeIndexBefore const):
  • Source/WebCore/platform/graphics/PlatformTimeRanges.h:

Canonical link: https://commits.webkit.org/266355@main

https://bugs.webkit.org/show_bug.cgi?id=258866

Reviewed by Xabier Rodriguez-Calvar.

Before this change, finding the place where new range should be added/merged/created was done
with linear approach.
This change provides logarithmic approach to find the place for new range.

* Source/WebCore/platform/graphics/PlatformTimeRanges.cpp:
(WebCore::PlatformTimeRanges::add):
(WebCore::PlatformTimeRanges::findLastRangeIndexBefore const):
* Source/WebCore/platform/graphics/PlatformTimeRanges.h:

Canonical link: https://commits.webkit.org/266355@main
@eocanha eocanha added upstream Related to an upstream bug (or should be at some point) wpe-2.38 labels Aug 4, 2023
@eocanha eocanha merged commit 96d9000 into wpe-2.38 Aug 4, 2023
@eocanha
Copy link
Copy Markdown
Member

eocanha commented Aug 4, 2023

Commits coming from upstream don't need much review. Merging.

(But please, next time add "Co-authored by Asurdej", because this patch is partially based on #1073)

@pgorszkowski-igalia pgorszkowski-igalia deleted the pgorszkowski/wpe-2.38/improve_range_creation branch September 4, 2023 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

upstream Related to an upstream bug (or should be at some point) wpe-2.38

Development

Successfully merging this pull request may close these issues.

2 participants