Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(149)

Side by Side Diff: src/object-observe.js

Issue 1065863003: Use array literals instead of array constructor in native javascript. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: rebase and fix Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/messages.js ('k') | src/regexp.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 "use strict"; 5 "use strict";
6 6
7 // Overview: 7 // Overview:
8 // 8 //
9 // This file contains all of the routing and accounting for Object.observe. 9 // This file contains all of the routing and accounting for Object.observe.
10 // User code will interact with these mechanisms via the Object.observe APIs 10 // User code will interact with these mechanisms via the Object.observe APIs
(...skipping 576 matching lines...) Expand 10 before | Expand all | Expand 10 after
587 if (!IS_NULL(pendingObservers)) { 587 if (!IS_NULL(pendingObservers)) {
588 SetPendingObservers(null); 588 SetPendingObservers(null);
589 for (var i in pendingObservers) { 589 for (var i in pendingObservers) {
590 CallbackDeliverPending(pendingObservers[i]); 590 CallbackDeliverPending(pendingObservers[i]);
591 } 591 }
592 } 592 }
593 } 593 }
594 594
595 function SetupObjectObserve() { 595 function SetupObjectObserve() {
596 %CheckIsBootstrapping(); 596 %CheckIsBootstrapping();
597 InstallFunctions($Object, DONT_ENUM, $Array( 597 InstallFunctions($Object, DONT_ENUM, [
598 "deliverChangeRecords", ObjectDeliverChangeRecords, 598 "deliverChangeRecords", ObjectDeliverChangeRecords,
599 "getNotifier", ObjectGetNotifier, 599 "getNotifier", ObjectGetNotifier,
600 "observe", ObjectObserve, 600 "observe", ObjectObserve,
601 "unobserve", ObjectUnobserve 601 "unobserve", ObjectUnobserve
602 )); 602 ]);
603 InstallFunctions($Array, DONT_ENUM, $Array( 603 InstallFunctions($Array, DONT_ENUM, [
604 "observe", ArrayObserve, 604 "observe", ArrayObserve,
605 "unobserve", ArrayUnobserve 605 "unobserve", ArrayUnobserve
606 )); 606 ]);
607 InstallFunctions(notifierPrototype, DONT_ENUM, $Array( 607 InstallFunctions(notifierPrototype, DONT_ENUM, [
608 "notify", ObjectNotifierNotify, 608 "notify", ObjectNotifierNotify,
609 "performChange", ObjectNotifierPerformChange 609 "performChange", ObjectNotifierPerformChange
610 )); 610 ]);
611 } 611 }
612 612
613 SetupObjectObserve(); 613 SetupObjectObserve();
OLDNEW
« no previous file with comments | « src/messages.js ('k') | src/regexp.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698