MediaStreamTrackSourcesRequest::create should take a const String&, not a String

As requested in comments on https://codereview.chromium.org/84923004/

BUG=297477

Review URL: https://codereview.chromium.org/113783004

git-svn-id: svn://svn.chromium.org/blink/trunk@163875 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove redundant test: COMPILER(GCC) || COMPILER(CLANG) => COMPILER(GCC)

COMPILER(CLANG) =implies=> COMPILER(GCC)
so
COMPILER(GCC) || COMPILER(CLANG) == COMPILER(GCC)

Remove redundant and misleading test.

Came up in:
[POSSIBLE PERFORMANCE IMPACT] Use __builtin_expect on Clang as well (for UNLIKELY and LIKELY)
https://codereview.chromium.org/114933002/
NOTRY=true

Review URL: https://codereview.chromium.org/105673006

git-svn-id: svn://svn.chromium.org/blink/trunk@163876 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move outgoing referrer/origin tracking to Document.

The outgoing referrer is, generally speaking, the url of the Document that
initiated the request (if such a Document exists). This patch codifies the
fetch spec's rules that the referrer should be omitted when the referrer would
be a url with scheme about:, data:, or javascript:.

The primary behavior change is how referrers are handled in the case where
document.open() is called by a different document. In that case, we currently
change Document::m_url to the url of the document that triggered the open(), but
we don't update the referrer. This patch unifies those variables, so we will now
update the outgoing referrer as well as the document url.

I believe the fetch standard implies this is the correct behavior. Firefox and
Safari match our old behavior, Presto-based Opera matches the new behavior.

BUG=

Review URL: https://codereview.chromium.org/114523002

git-svn-id: svn://svn.chromium.org/blink/trunk@163877 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Simplify determining CachePolicy

Merge the subresource and main resource paths and put them in FrameFetchContext.
They don't really need to be separate as far as I can tell, and it's a moot
point at the moment anyway, since we never cache main resources.

Remove CachePolicyCache, which is unused.

Remove the CachePolicy parameter from Resource::mustRevalidateDueToCacheHeaders.
The usage of CachePolicy in that function is simple enough to inline in
ResourceFetcher, and it means the function name is misleading, since we are
currently making a decision based on more than just the cache headers.

BUG=none

Review URL: https://codereview.chromium.org/114523004

git-svn-id: svn://svn.chromium.org/blink/trunk@163878 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Followup from r163843. I missed a line when relanding the change.

BUG=304341
R=jochen@chromium.org

Review URL: https://codereview.chromium.org/115073002

git-svn-id: svn://svn.chromium.org/blink/trunk@163879 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert of https://codereview.chromium.org/100723003/
Reason for revert: Performance regression in calls to getAttribute.

BUG=328347
TBR=ojan@chromium.org,eseidel@chromium.org
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/115403002

git-svn-id: svn://svn.chromium.org/blink/trunk@163880 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Remove assertion when removing security origin.

Not that frames could be attached and navigated separatelyit is possible that we navigate/detach a frame
that does not have any security origin (it was attached but never navigated).

Drive-by: Add console.assert hook to inspector-test.js to catch incorrect assertions such problems in future.
Drive-by: Fix annother test that now fails due to incorrect assertion: baseURL is not mandatory on document.
E.g. baseURL might be null when document was created programmatically.

R=pfeldman_ooo, apavlov, eustas

Review URL: https://codereview.chromium.org/100223005

git-svn-id: svn://svn.chromium.org/blink/trunk@163881 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Crash in RawResource::responseReceived() on successful revalidation.

BUG=247778
TEST=RawResourceTest.RevalidationSucceeded

Review URL: https://codereview.chromium.org/103123003

git-svn-id: svn://svn.chromium.org/blink/trunk@163882 bbb929c8-8fbe-4397-9dbb-9b2b20218538