Title: [264611] trunk/Source/WebCore
- Revision
- 264611
- Author
- [email protected]
- Date
- 2020-07-20 12:58:33 -0700 (Mon, 20 Jul 2020)
Log Message
Streamline URLDecomposition::setHash
https://bugs.webkit.org/show_bug.cgi?id=214561
Reviewed by Anders Carlsson.
* html/URLDecomposition.cpp:
(WebCore::URLDecomposition::setHash): No need to cast to StringView since we
already have a StringView. Also don't need a local variable.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (264610 => 264611)
--- trunk/Source/WebCore/ChangeLog 2020-07-20 19:52:09 UTC (rev 264610)
+++ trunk/Source/WebCore/ChangeLog 2020-07-20 19:58:33 UTC (rev 264611)
@@ -1,5 +1,16 @@
2020-07-20 Darin Adler <[email protected]>
+ Streamline URLDecomposition::setHash
+ https://bugs.webkit.org/show_bug.cgi?id=214561
+
+ Reviewed by Anders Carlsson.
+
+ * html/URLDecomposition.cpp:
+ (WebCore::URLDecomposition::setHash): No need to cast to StringView since we
+ already have a StringView. Also don't need a local variable.
+
+2020-07-20 Darin Adler <[email protected]>
+
[Cocoa] MediaSampleAVFObjC::createImageSample depends on function argument evaluation order
https://bugs.webkit.org/show_bug.cgi?id=214560
Modified: trunk/Source/WebCore/html/URLDecomposition.cpp (264610 => 264611)
--- trunk/Source/WebCore/html/URLDecomposition.cpp 2020-07-20 19:52:09 UTC (rev 264610)
+++ trunk/Source/WebCore/html/URLDecomposition.cpp 2020-07-20 19:58:33 UTC (rev 264611)
@@ -241,10 +241,8 @@
auto fullURL = this->fullURL();
if (value.isEmpty())
fullURL.removeFragmentIdentifier();
- else {
- auto newFragment = value.startsWith('#') ? StringView(value).substring(1) : StringView(value);
- fullURL.setFragmentIdentifier(newFragment);
- }
+ else
+ fullURL.setFragmentIdentifier(value.startsWith('#') ? value.substring(1) : value);
setFullURL(fullURL);
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
